Akanksha-kedia commented on PR #6760: URL: https://github.com/apache/hive/pull/6760#issuecomment-5598881943
Correction: my first fix (\`System.setProperty(...)\` in \`@BeforeClass\`) did **not** actually work — Jenkins re-ran and hit the identical \`SecurityException\` at the shifted line number, confirming it. \`ClassSecurityValidator\` reads its trusted-classes allowlist once when it's first loaded, not per-call, and something in the same Surefire fork can trigger that load before \`@BeforeClass\` runs — so setting it at test runtime is too late regardless of which method it's in. Fixed properly now: moved the trust declaration into \`kafka-handler/pom.xml\`'s own \`maven-surefire-plugin\` \`systemPropertyVariables\`, so it's present as a JVM property from fork startup. Reverted the ineffective test-file change. (Separately, the \`TestLlapZookeeperRegistryImpl\` failure in the same run is under Jenkins' own "Existing failures" bucket — pre-existing, unrelated to this change.) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
