You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Riza Suminto (Code Review)" <ge...@cloudera.org> on 2023/11/01 01:14:30 UTC

[Impala-ASF-CR] IMPALA-12508: Update query option "enabled runtime filter types" comments

Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/20627 )

Change subject: IMPALA-12508: Update query option "enabled_runtime_filter_types" comments
......................................................................


Patch Set 4: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/20627/3/common/thrift/ImpalaService.thrift
File common/thrift/ImpalaService.thrift:

http://gerrit.cloudera.org:8080/#/c/20627/3/common/thrift/ImpalaService.thrift@540
PS3, Line 540:   //     ALL     - apply all type
> Zihao Ye's interpretation matches my reading of assignRuntimeFilters as wel
That is correct. Thanks!



-- 
To view, visit http://gerrit.cloudera.org:8080/20627
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iecff7c655e273712d851d9ce94ef5713e579aa72
Gerrit-Change-Number: 20627
Gerrit-PatchSet: 4
Gerrit-Owner: Zihao Ye <ey...@163.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Zihao Ye <ey...@163.com>
Gerrit-Comment-Date: Wed, 01 Nov 2023 01:14:30 +0000
Gerrit-HasComments: Yes