davisusanibar commented on code in PR #229:
URL: https://github.com/apache/arrow-cookbook/pull/229#discussion_r923973844
##########
java/source/demo/pom.xml:
##########
@@ -21,9 +21,17 @@
<properties>
<maven.compiler.source>8</maven.compiler.source>
<maven.compiler.target>8</maven.compiler.target>
- <arrow.version>8.0.0</arrow.version>
+ <arrow.version>9.0.0.dev191</arrow.version><!-- Prepare for 9.0.0
release -->
Review Comment:
> IMO it's fine to only test Java 11 (especially if that's the language
level we target) (or, compile with 11 and run on 17), and again: do we care
about stable/nightly cookbooks? That is a separate discussion
Ok. PoC about how to pass variables matrix from .ci github to Java cookbook
is possible, now we need to discuss: `do we care about stable/nightly
cookbooks? `
This is the current setup:
```
strategy:
matrix:
python-version: [ '3.9' ]
java-version: [ '11', '17', '18' ]
compiler-version: ['11']
arrow-version: ['9.0.0.dev405']
```
This could be some capabilities that is needed to implement if we need to
care about stable/dev:
- Configure skip/consider cookbook per Arrow Java version needed (for new
dev features that is not supported on stable version)
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
