Skip to content

Conversation

@ChangRui-Ryan
Copy link

What problem does this PR solve?

Issue Number: close #xxx

Problem Summary:

What is changed and how it works?

Proposal: xxx

What's Changed:

How it Works:

Related changes

  • PR to update pingcap/docs/pingcap/docs-cn:
  • Need to cherry-pick to the release branch

@ChangRui-Ryan
Copy link
Author

/retest

@ChangRui-Ryan
Copy link
Author

/retest

@ChangRui-Ryan
Copy link
Author

@windtalker @wshwsh12 @winoros PTAL, thanks

@windtalker
Copy link
Contributor

/retest

Copy link
Contributor

@windtalker windtalker left a comment

Choose a reason for hiding this comment

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

lgtm

// because upper executors will have a TopK anyway.
// If not specified (optional, or top_k is very large), then all results will be returned.
// This option is not supported for QueryType == NoScore.
// These fields support pushdown for order-by and limit scenarios.
Copy link
Contributor

Choose a reason for hiding this comment

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

why remove the original comments?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants