You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@orc.apache.org by do...@apache.org on 2021/08/11 06:10:52 UTC
[orc] branch main updated: ORC-934: Add integration tests for Java
bench (#848)
This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/orc.git
The following commit(s) were added to refs/heads/main by this push:
new f1ac372 ORC-934: Add integration tests for Java bench (#848)
f1ac372 is described below
commit f1ac372db11c8a0c5fbfcfb7905f14609b8cd4d1
Author: William Hyun <wi...@apache.org>
AuthorDate: Tue Aug 10 23:10:48 2021 -0700
ORC-934: Add integration tests for Java bench (#848)
### What changes were proposed in this pull request?
This PR aims to add integration tests for Java bench.
### Why are the changes needed?
To prevent further regressions.
### How was this patch tested?
Pass the CIs.
---
java/CMakeLists.txt | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
diff --git a/java/CMakeLists.txt b/java/CMakeLists.txt
index 231bf11..130134b 100644
--- a/java/CMakeLists.txt
+++ b/java/CMakeLists.txt
@@ -64,6 +64,25 @@ add_test(
COMMAND java -jar tools/orc-tools-${ORC_VERSION}-uber.jar version
WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR})
+execute_process(
+ COMMAND java --add-opens java.base/java.nio=ALL-UNNAMED -version
+ WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
+ RESULT_VARIABLE RET)
+if(RET EQUAL 0)
+ set(ADD_OPENS --add-opens)
+ set(JAVA_NIO java.base/java.nio=ALL-UNNAMED)
+endif()
+
+add_test(
+ NAME java-bench-gen-test
+ COMMAND java ${ADD_OPENS} ${JAVA_NIO} -jar bench/core/orc-benchmarks-core-${ORC_VERSION}-uber.jar generate data -d sales -s 1
+ WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR})
+
+add_test(
+ NAME java-bench-scan-test
+ COMMAND java ${ADD_OPENS} ${JAVA_NIO} -jar bench/core/orc-benchmarks-core-${ORC_VERSION}-uber.jar scan data -d sales
+ WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR})
+
install(
FILES ${ORC_JARS}
DESTINATION share)