Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/24448 )
Change subject: IMPALA-14799: Add oauth_servers support and tests ...................................................................... Patch Set 4: (5 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/4/be/src/service/impala-server.cc File be/src/service/impala-server.cc: http://gerrit.cloudera.org:8080/#/c/24448/4/be/src/service/impala-server.cc@3237 PS4, 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/4/be/src/util/oauth-server-config-test.cc File be/src/util/oauth-server-config-test.cc: http://gerrit.cloudera.org:8080/#/c/24448/4/be/src/util/oauth-server-config-test.cc@127 PS4, Line 127: TEST(OAuthServerConfigTest, BuildOAuthServerConfigsMergesLegacyJwtFlagsFromUsernameClaim) { line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/24448/4/be/src/util/oauth-server-config.cc File be/src/util/oauth-server-config.cc: http://gerrit.cloudera.org:8080/#/c/24448/4/be/src/util/oauth-server-config.cc@120 PS4, Line 120: Status ParseOAuthServerObject(const rapidjson::Value& obj, OAuthServerConfig* config_out) { line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/24448/4/be/src/util/oauth-server-config.cc@125 PS4, Line 125: RETURN_IF_ERROR(ReadOptionalStringField(obj, "caCertFilePath", &config.ca_cert_file_path)); line too long (93 > 90) http://gerrit.cloudera.org:8080/#/c/24448/4/be/src/util/oauth-server-config.cc@126 PS4, Line 126: 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: 4 Gerrit-Owner: Anubhav Jindal <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Fri, 12 Jun 2026 00:53:11 +0000 Gerrit-HasComments: Yes
