Surya Hebbar has posted comments on this change. ( http://gerrit.cloudera.org:8080/23154 )
Change subject: IMPALA-9846: Enable Aggregated Runtime Profile by Default ...................................................................... Patch Set 22: (1 comment) With these changes, the shift to aggregated runtime profile is finally complete. Also, while transitioning the many possible 'row_regex' tests into the more sustainable 'aggregation(SUM)' tests. These changes have been rigorously tested with multiple builds. And with the following final successful build, all the test have been fully fixed. https://master-03.jenkins.cloudera.com/job/impala-private-basic-parameterized/2965/ http://gerrit.cloudera.org:8080/#/c/23154/22/common/thrift/RuntimeProfile.thrift File common/thrift/RuntimeProfile.thrift: http://gerrit.cloudera.org:8080/#/c/23154/22/common/thrift/RuntimeProfile.thrift@76 PS22, Line 76: 5: required TCounterCategory type > Adding a required field 'type' in TAggCounter might break the compatibility I will consider changing this to optional. -- To view, visit http://gerrit.cloudera.org:8080/23154 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If41d6322361fba82c946efd614cc7d28cb1c36e8 Gerrit-Change-Number: 23154 Gerrit-PatchSet: 22 Gerrit-Owner: Surya Hebbar <[email protected]> Gerrit-Reviewer: Abhishek Rawat <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Kurt Deschler <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Surya Hebbar <[email protected]> Gerrit-Comment-Date: Mon, 29 Dec 2025 21:34:34 +0000 Gerrit-HasComments: Yes
