MINOR: remove one duplicated inparam in TopK - Githubissues

apache / datafusion

Apache DataFusion SQL Query Engine

https://datafusion.apache.org/

Apache License 2.0

6.33k stars 1.2k forks source link

MINOR: remove one duplicated inparam in TopK #13479

Closed waynexia closed 2 days ago

waynexia commented 2 days ago

Which issue does this PR close?

Closes #.

Rationale for this change

I came across this when writing https://github.com/GreptimeTeam/greptimedb/pull/5018. partition and partition_id are the same things.

What changes are included in this PR?

A minor refactor to remove the unnecessary inparam.

Are these changes tested?

No need

Are there any user-facing changes?

API change for those who use TopK directly like me in https://github.com/GreptimeTeam/greptimedb/pull/5018