rymurr commented on a change in pull request #7289:
URL: https://github.com/apache/arrow/pull/7289#discussion_r433132611
##########
File path: dev/archery/archery/integration/tester_java.py
##########
@@ -37,7 +37,8 @@ class JavaTester(Tester):
FLIGHT_SERVER = True
FLIGHT_CLIENT = True
- JAVA_OPTS = ['-Dio.netty.tryReflectionSetAccessible=true']
+ JAVA_OPTS = ['-Dio.netty.tryReflectionSetAccessible=true',
+ '-Darrow.struct.conflict.policy=CONFLICT_APPEND']
Review comment:
Hey @lidavidm, yes the 'compatible' behaviour is opt-in. This may be
controversial but through a mix of opinionated, ignorant and lazy I don't
understand the value of duplicate names in a struct. Given the scope of
implementing such a change fully and the unintended consequences downstream I
have opted to give the library user the option to be compatible with the c++
IPC or maintain backwards compatibility with Java. I am happy to hear what the
community thinks, especially if this approach is seen as too heavy handed.
This flag wasn't document anywhere so I have added a note to the 'Java
Properties' section in the Java README.md.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
