Skip to content

Add comment for AddDistinctForSemiJoinBuild optimizer#25254

Merged
feilong-liu merged 1 commit intoprestodb:masterfrom
feilong-liu:add_comment_2
Jun 6, 2025
Merged

Add comment for AddDistinctForSemiJoinBuild optimizer#25254
feilong-liu merged 1 commit intoprestodb:masterfrom
feilong-liu:add_comment_2

Conversation

@feilong-liu
Copy link
Contributor

Description

add comment to optimizer added in #25238

Motivation and Context

Impact

Test Plan

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

Please follow release notes guidelines and fill in the release notes below.

If release note is NOT required, use:

== NO RELEASE NOTE ==

@feilong-liu feilong-liu requested a review from a team as a code owner June 5, 2025 19:36
@feilong-liu feilong-liu requested a review from hantangwangd June 5, 2025 19:36
@prestodb-ci prestodb-ci added the from:Meta PR from Meta label Jun 5, 2025
@feilong-liu feilong-liu requested a review from jaystarshot June 5, 2025 19:37
Copy link
Member

@hantangwangd hantangwangd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding this comment.

@feilong-liu feilong-liu merged commit 1d46dc8 into prestodb:master Jun 6, 2025
98 checks passed
@feilong-liu feilong-liu deleted the add_comment_2 branch June 6, 2025 04:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

from:Meta PR from Meta

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants