Re: [VOTE] JanusGraph 0.5.0 release
Oleksandr Porunov <alexand...@...>
Thank you Ted for the review of the release! Yes, I also noticed that now jdk is included into elasticsearch. This change was made in the next commit: https://github.com/JanusGraph/janusgraph/commit/c16f5c28c859f1ab209e344a1a83ee4f07038c8f Before that commit we were executing assembly tests against ElasticSearch 6.2.4 After that commit we changed the behavior and now we are executing assembly tests against ElasticSearch {elasticsearch.version} (which is 7.6.1 right now). ES 7.x depends on java 13, so I assume that is why they included openjdk 13 into the elasticsearch 7.x releases. I am not sure if we can exclude jdk 13 from the distribution now. I think it should be there now to work with newer ES releases. But this dependency is only for distribution. It isn't populated into maven, so I guess it is OK. On Friday, March 13, 2020 at 9:18:15 AM UTC-7, Ted Wilmes wrote:
Thanks for leading this release Oleksandr! |
|