Skip to content

Commit 6ca5d7c

Browse files
test(tags): recalibrate endpoint-add pin to 94 after Stage 2 bulk-propagate
The earlier Stage 2 commit recorded 194 queries for product-tag add propagating to 100 endpoints, before bulk_propagate_inherited_tags, the thread-local batch context, and the Location target-name precompute landed on this branch. With those in place the actual count is 94. Pinned value updated; comment refreshed.
1 parent bbd571c commit 6ca5d7c

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

unittests/test_tag_inheritance_perf.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -380,10 +380,10 @@ def test_baseline_product_tag_remove_propagates_to_100_locations_v3(self):
380380
# Phase B Stage 2 raises endpoint add 91 -> 194 because the eager Celery
381381
# propagate dispatched by m2m_changed and the explicit
382382
# propagate_tags_on_product_sync call both pay the new tags-read for
383-
# bulk re-merge. Acceptable: the same lever delivers a 27% drop on the
384-
# ZAP import path. Will go further down in Stages 3+4+5 when the
385-
# duplicate inherited_tags M2M is dropped.
386-
EXPECTED_PRODUCT_TAG_ADD_100_ENDPOINTS = 194
383+
# bulk re-merge. The bulk-propagate + batch-context + Location precompute
384+
# commits later in Stage 2 collapse this back to 94. Will go further down
385+
# in Stages 3+4+5 when the duplicate inherited_tags M2M is dropped.
386+
EXPECTED_PRODUCT_TAG_ADD_100_ENDPOINTS = 94
387387
EXPECTED_PRODUCT_TAG_REMOVE_100_ENDPOINTS = 56
388388

389389
# V3 location paths. Pre-Phase-A: 4532 add, 4307 remove.

0 commit comments

Comments
 (0)