Skip to content

Commit 19f506d

Browse files
committed
[BugFix] Drop issue-link reference from regression-test JavaDoc (#5488)
Per maintainer review feedback, the regression-test JavaDoc for testDedupAfterWhereProducesLogicalDedup mentioned the originating issue URL. The remaining JavaDoc paragraphs already describe the bug shape and the rule-ordering invariant, so the explicit issue link is unnecessary noise. Signed-off-by: ryan-gh-bot <ryan-gh-bot@users.noreply.github.com>
1 parent 8bfb649 commit 19f506d

1 file changed

Lines changed: 1 addition & 3 deletions

File tree

ppl/src/test/java/org/opensearch/sql/ppl/calcite/CalcitePPLDedupTest.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -361,9 +361,7 @@ public void testSortFieldProjectedAwayBeforeDedup() {
361361
}
362362

363363
/**
364-
* Regression test for https://github.com/opensearch-project/sql/issues/5482
365-
*
366-
* <p>When a user {@code where} precedes {@code dedup}, the user filter sits adjacent to the
364+
* When a user {@code where} precedes {@code dedup}, the user filter sits adjacent to the
367365
* bucket-non-null filter that PPL emits for the dedup pattern. The HEP optimizer must run {@link
368366
* PPLSimplifyDedupRule} before {@link FilterMergeRule}; otherwise the merged condition breaks the
369367
* simplify-rule's operand match and dedup falls through to the in-memory {@code ROW_NUMBER}

0 commit comments

Comments
 (0)