Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20985 )
Change subject: IMPALA-12780: Only show non-default options in the catalog operations page ...................................................................... IMPALA-12780: Only show non-default options in the catalog operations page The details shown in the catalog operations page are too verbose. For instance, for ExecDdlRequest, we show the string of TDdlQueryOptions: query_options=TDdlQueryOptions(sync_ddl:false, debug_action:, lock_max_wait_time_s:300, kudu_table_reserve_seconds:0) What really matters is the non-default options, e.g. if sync_ddl is set to true, it should be shown. This patch improves the details field to show only non-default options. Also wraps the content of Query Id and Details columns so they can fit into the table. Change-Id: Ie62d4b9e4d357e02764e7a62f4dc107de602e1a5 Reviewed-on: http://gerrit.cloudera.org:8080/20985 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M common/thrift/CatalogService.thrift M fe/src/main/java/org/apache/impala/catalog/monitor/CatalogOperationTracker.java M www/catalog_operations.tmpl 3 files changed, 55 insertions(+), 22 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/20985 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie62d4b9e4d357e02764e7a62f4dc107de602e1a5 Gerrit-Change-Number: 20985 Gerrit-PatchSet: 4 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]>
