ayushtkn commented on code in PR #6760:
URL: https://github.com/apache/hive/pull/6760#discussion_r3978201289
##########
kafka-handler/pom.xml:
##########
@@ -227,6 +227,20 @@
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-jar-plugin</artifactId>
</plugin>
+ <plugin>
+ <groupId>org.apache.maven.plugins</groupId>
+ <artifactId>maven-surefire-plugin</artifactId>
+ <configuration>
+ <!-- Avro's ClassSecurityValidator reads the trusted-classes
allowlist once when it is
+ first loaded, so it must be present as a JVM system property
from fork startup;
+ setting it at test runtime (e.g. in @BeforeClass) is too late.
Scoped to this
+ module's own SimpleRecord test fixture (used by
AvroBytesConverterTest), not a
+ repo-wide trust grant. -->
+ <systemPropertyVariables>
+
<org.apache.avro.SERIALIZABLE_CLASSES>org.apache.hadoop.hive.kafka.SimpleRecord</org.apache.avro.SERIALIZABLE_CLASSES>
+ </systemPropertyVariables>
+ </configuration>
Review Comment:
Not very straight forward, do you have idea what introduced this? I am even
not sure if this is only test only fix and blast during runtime, did you
reproduced it locally and verified it works by this change? I doubt
--
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]
