zzzxl1993 commented on code in PR #43643:
URL: https://github.com/apache/doris/pull/43643#discussion_r1838012741
##########
regression-test/suites/inverted_index_p0/test_index_approx_top_k.groovy:
##########
@@ -88,15 +88,11 @@ suite("test_index_approx_top_k", "p0"){
sql """ set debug_skip_fold_constant = true; """
qt_sql """ select clientip, count(*) as count from ${tableName} group
by clientip order by count desc, clientip asc limit 10; """
- qt_sql """ select approx_top_k(clientip) from ${tableName}; """
- qt_sql """ select approx_top_k(clientip, 10) from ${tableName}; """
Review Comment:
Currently, it is difficult to implement default values for the variable
arguments in the function framework. It is hoped that users will directly
specify the last two parameters.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
