Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/21775 )
Change subject: IMPALA-13372: Declare batch_size dimension for count_star tests ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/21775/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/21775/1//COMMIT_MSG@12 PS1, Line 12: dimension and and let vector permutation run the test twice for typo: and and http://gerrit.cloudera.org:8080/#/c/21775/1//COMMIT_MSG@17 PS1, Line 17: in test_min_multiple_distinct are dropped. cluster_sizes parameter in The cluster_sizes parameter was meant to be something tests could incrementally opt into as we expanded tests to cover ALL_CLUSTER_SIZES. I'm not sure I understand the rationale for removing it. http://gerrit.cloudera.org:8080/#/c/21775/1//COMMIT_MSG@20 PS1, Line 20: are droped and replaced by declaration using typo: droped -- To view, visit http://gerrit.cloudera.org:8080/21775 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1b0399ee3b39a5b0bbfdf8a05680ab1fb23d8f87 Gerrit-Change-Number: 21775 Gerrit-PatchSet: 1 Gerrit-Owner: Riza Suminto <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Yida Wu <[email protected]> Gerrit-Comment-Date: Mon, 09 Sep 2024 23:48:51 +0000 Gerrit-HasComments: Yes
