Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/24448 )
Change subject: IMPALA-14799: Add oauth_servers JSON config for multiple identity providers ...................................................................... Patch Set 1: (4 comments) gerrit-auto-critic failed. You can reproduce it locally using command: python3 bin/jenkins/critique-gerrit-review.py --dryrun To run it, you might need a virtual env with Python3's venv installed. http://gerrit.cloudera.org:8080/#/c/24448/1/be/src/service/impala-server.cc File be/src/service/impala-server.cc: http://gerrit.cloudera.org:8080/#/c/24448/1/be/src/service/impala-server.cc@3237 PS1, Line 3237: const bool require_oauth_servers = (FLAGS_jwt_token_auth && FLAGS_jwt_validate_signature) line too long (93 > 90) http://gerrit.cloudera.org:8080/#/c/24448/1/be/src/util/oauth-server-config.cc File be/src/util/oauth-server-config.cc: http://gerrit.cloudera.org:8080/#/c/24448/1/be/src/util/oauth-server-config.cc@104 PS1, Line 104: Status ParseOAuthServerObject(const rapidjson::Value& obj, OAuthServerConfig* config_out) { line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/24448/1/be/src/util/oauth-server-config.cc@109 PS1, Line 109: RETURN_IF_ERROR(ReadOptionalStringField(obj, "caCertFilePath", &config.ca_cert_file_path)); line too long (93 > 90) http://gerrit.cloudera.org:8080/#/c/24448/1/be/src/util/oauth-server-config.cc@110 PS1, Line 110: RETURN_IF_ERROR(ReadOptionalBoolField(obj, "verifyServerCert", &config.verify_server_cert)); line too long (94 > 90) -- To view, visit http://gerrit.cloudera.org:8080/24448 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib29ff36600406ba59c10f29d79cc632020f4a3f7 Gerrit-Change-Number: 24448 Gerrit-PatchSet: 1 Gerrit-Owner: Anubhav Jindal <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Fri, 12 Jun 2026 00:44:16 +0000 Gerrit-HasComments: Yes
