pitrou commented on code in PR #13614:
URL: https://github.com/apache/arrow/pull/13614#discussion_r926353925
##########
java/dataset/src/main/cpp/CMakeLists.txt:
##########
@@ -63,3 +63,5 @@ add_arrow_test(dataset_jni_test
jni_util.cc
EXTRA_INCLUDES
${JNI_INCLUDE_DIRS})
+
+install(FILES jni_util.h DESTINATION "${CMAKE_INSTALL_INCLUDEDIR}/jni/dataset")
Review Comment:
In any case, there is some ongoing work on the JNI layer to refactor it,
base it on the C Data Interface, and try to minimize the amount of custom (and
poorly-tested) code. So these are likely to break again.
--
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]
