[GitHub] [arrow] wesm commented on pull request #7060: ARROW-8619: [C++] Use distinct enum values for MonthInterval, DayTimeInterval

2020-04-30 Thread GitBox 


wesm commented on pull request #7060:
URL: https://github.com/apache/arrow/pull/7060#issuecomment-622156273


   The ursabot build failures are spurious



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:
us...@infra.apache.org

[GitHub] [arrow] wesm commented on pull request #7060: ARROW-8619: [C++] Use distinct enum values for MonthInterval, DayTimeInterval

2020-04-30 Thread GitBox 


wesm commented on pull request #7060:
URL: https://github.com/apache/arrow/pull/7060#issuecomment-621933237


   For some reason I can't get JNI running in my local setup
   
   ```
   CMake Error at 
/home/wesm/cpp-toolchain/share/cmake-3.16/Modules/FindPackageHandleStandardArgs.cmake:146
 (message):
 Could NOT find JNI (missing: JAVA_AWT_INCLUDE_PATH)
   Call Stack (most recent call first):
 
/home/wesm/cpp-toolchain/share/cmake-3.16/Modules/FindPackageHandleStandardArgs.cmake:393
 (_FPHSA_FAILURE_MESSAGE)
 /home/wesm/cpp-toolchain/share/cmake-3.16/Modules/FindJNI.cmake:372 
(FIND_PACKAGE_HANDLE_STANDARD_ARGS)
 src/gandiva/jni/CMakeLists.txt:27 (find_package)
   ```
   
   tried with 
   
   ```
   export JAVA_HOME=/usr/lib/jvm/java-8-openjdk-amd64
   ```
   
   and 
   
   ```
   export JAVA_HOME=/usr/lib/jvm/java-11-openjdk-amd64
   ```
   
   It worked with
   
   ```
   export JAVA_HOME=/usr/lib/jvm/java-14-oracle
   ```
   
   though



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:
us...@infra.apache.org

[GitHub] [arrow] wesm commented on pull request #7060: ARROW-8619: [C++] Use distinct enum values for MonthInterval, DayTimeInterval

2020-04-30 Thread GitBox 


wesm commented on pull request #7060:
URL: https://github.com/apache/arrow/pull/7060#issuecomment-621872082


   Will fix the JNI issue and will notify the mailing list



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:
us...@infra.apache.org

[GitHub] [arrow] wesm commented on pull request #7060: ARROW-8619: [C++] Use distinct enum values for MonthInterval, DayTimeInterval

2020-04-29 Thread GitBox 


wesm commented on pull request #7060:
URL: https://github.com/apache/arrow/pull/7060#issuecomment-621472148


   Will fix the builds



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:
us...@infra.apache.org