steveloughran commented on code in PR #7128:
URL: https://github.com/apache/hadoop/pull/7128#discussion_r1816391775
##########
hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/TestText.java:
##########
@@ -344,6 +344,8 @@ public void testConcurrentEncodeDecode() throws Exception{
@Test
public void testAvroReflect() throws Exception {
+ // Avro expects explicitely stated, trusted packages used for
(de-)serialization
+ System.setProperty("org.apache.avro.SERIALIZABLE_PACKAGES",
"org.apache.hadoop.io.Text");
Review Comment:
see comment above
##########
hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestPath.java:
##########
@@ -404,6 +404,8 @@ public void testGetName() {
@Test (timeout = 30000)
public void testAvroReflect() throws Exception {
+ // Avro expects explicitely stated, trusted packages used for
(de-)serialization
+ System.setProperty("org.apache.avro.SERIALIZABLE_PACKAGES",
"org.apache.hadoop.fs.Path");
Review Comment:
make the property name a constant somewhere in hadoop-common tests and refer
to it in both places
--
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]
