gaurav singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/22424 )
Change subject: IMPALA-13675: OAuth AuthN Support for Impala Shell ...................................................................... Patch Set 41: (2 comments) http://gerrit.cloudera.org:8080/#/c/22424/40/shell/impala_shell/option_parser.py File shell/impala_shell/option_parser.py: http://gerrit.cloudera.org:8080/#/c/22424/40/shell/impala_shell/option_parser.py@378 PS40, Line 378: help=SUPPRESS_HELP) > Was this added by accident? Doesn't seem like it should be here. Its intentional. Mentioned in the git commit message: Also suppressed existing option hs2_x_forward from the impala --help output. http://gerrit.cloudera.org:8080/#/c/22424/40/tests/custom_cluster/test_shell_jwt_auth.py File tests/custom_cluster/test_shell_jwt_auth.py: http://gerrit.cloudera.org:8080/#/c/22424/40/tests/custom_cluster/test_shell_jwt_auth.py@57 PS40, Line 57: > Need to remove these OAUTH defs since the tests were moved to test_shell_oa Done -- To view, visit http://gerrit.cloudera.org:8080/22424 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I84e26d54f6a53696660728efb239ffd43de4c55d Gerrit-Change-Number: 22424 Gerrit-PatchSet: 41 Gerrit-Owner: gaurav singh <[email protected]> Gerrit-Reviewer: Abhishek Rawat <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: gaurav singh <[email protected]> Gerrit-Comment-Date: Wed, 04 Jun 2025 20:18:58 +0000 Gerrit-HasComments: Yes
