Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/20853 )
Change subject: IMPALA-12670: getIfPresent should throw the cause of error ...................................................................... Patch Set 3: Code-Review+1 (2 comments) http://gerrit.cloudera.org:8080/#/c/20853/2/be/src/exec/catalog-op-executor.cc File be/src/exec/catalog-op-executor.cc: http://gerrit.cloudera.org:8080/#/c/20853/2/be/src/exec/catalog-op-executor.cc@60 PS2, Line 60: DEFINE_double_hidden(inject_failure_ratio_in_catalog_fetch, -1, > Yeah, I think either way is ok. It's a hidden flag and we don't plan to exp Ack http://gerrit.cloudera.org:8080/#/c/20853/2/be/src/exec/catalog-op-executor.cc@61 PS2, Line 61: "Ratio to randomly fail the GetPartialCatalogObject RPC with TABLE_NOT_LOADED " > Any value is allowed. Negative values disable it. Values larger than 1 alwa Done -- To view, visit http://gerrit.cloudera.org:8080/20853 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I74268ba2bb700988107780e13ffbdbb4c767d09d Gerrit-Change-Number: 20853 Gerrit-PatchSet: 3 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Fri, 05 Jan 2024 00:30:43 +0000 Gerrit-HasComments: Yes
