Skip to content

Reinstate "Optimize issuable updates"

Markus Koller requested to merge 21068-unrevert-optimize-issuable-updates into master

What does this MR do?

This reinstates !58114 (merged), which was reverted in !59355 (merged) due to causing a master:broken in #327668 (closed).

The fix for the failing spec was removing the unneeded stub_const usage in !59359 (merged).

The specs still pass in this MR when run in the order that was previously triggering the problem:

bin/rspec ee/spec/features/epics/epic_issues_spec.rb ee/spec/services/ee/issuable/bulk_update_service_spec.rb
bin/rspec ee/spec/services/epic_links/create_service_spec.rb ee/spec/services/ee/issuable/bulk_update_service_spec.rb

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team

Related to #21068

Edited by Markus Koller

Merge request reports