Quanlong Huang has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19555 )
Change subject: IMPALA-11953: Declare num_trues and num_falses in TIntermediateColumnStats as optional ...................................................................... IMPALA-11953: Declare num_trues and num_falses in TIntermediateColumnStats as optional TIntermediateColumnStats is the representation of incremental stats which are stored in HMS partition properties using keys like "impala_intermediate_stats_chunk0", "impala_intermediate_stats_chunk1", "impala_intermediate_stats_chunk2", etc. Fields in TIntermediateColumnStats should be optional to ensure backward compatibility. IMPALA-8205 adds two required fields, num_trues and num_falses, in TIntermediateColumnStats. This breaks the incremental stats loading in higher versions of Impala if the stats are generated by older Impala versions (< 4.0). This patch changes the fields to be optional. Tests: - Verified the incremental stats generated by CDH Impala cluster can be loaded by CDP Impala cluster with this fix. Change-Id: I4f74d5d0676e7ce9eb4ea8061a15610846db3ca5 Reviewed-on: http://gerrit.cloudera.org:8080/19555 Reviewed-by: Riza Suminto <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M common/thrift/CatalogObjects.thrift 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Riza Suminto: Looks good to me, approved Impala Public Jenkins: Verified -- 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: merged Gerrit-Change-Id: I4f74d5d0676e7ce9eb4ea8061a15610846db3ca5 Gerrit-Change-Number: 19555 Gerrit-PatchSet: 2 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]>
