Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/19555 )
Change subject: IMPALA-11953: Declare num_trues and num_falses in TIntermediateColumnStats as optional ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/19555/1/common/thrift/CatalogObjects.thrift File common/thrift/CatalogObjects.thrift: http://gerrit.cloudera.org:8080/#/c/19555/1/common/thrift/CatalogObjects.thrift@219 PS1, Line 219: // Estimated number of true and false value for boolean type : 5: required i64 num_trues : 6: required i64 num_falses These also added by IMPALA-8205. Should they turned to optional as well? -- To view, visit http://gerrit.cloudera.org:8080/19555 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4f74d5d0676e7ce9eb4ea8061a15610846db3ca5 Gerrit-Change-Number: 19555 Gerrit-PatchSet: 1 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Tue, 28 Feb 2023 01:03:14 +0000 Gerrit-HasComments: Yes
