Re: SpiUriDeploy TC job fails on Java 9+

2019-07-03 Thread Павлухин Иван
Folks, I made SpiUriDeploy job successful by enforcing jdk 8 and not passing any extra jvm arguments for an extra compile step (Step 4: Pre-compile external data). Everything is running fine so far [1]. [1] https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiUriDeploy&ta

Re: SpiUriDeploy TC job fails on Java 9+

2019-07-03 Thread Павлухин Иван
Ilya, Dmitriy, Are we going to drop only compilation on Java 9 and 10 but continue running tests on all versions (8-11)? The idea sounds ok to me. But to be honest I must say that currently we compile Ignite only using Java 8. Also it does not help me to fix SpiUriDeploy job. It has an extra comp

Re: SpiUriDeploy TC job fails on Java 9+

2019-07-03 Thread Dmitriy Pavlov
It was by mistake. Let's drop 9 && 10 compilation. ср, 3 июл. 2019 г. в 12:54, Ilya Kasnacheev : > Hello! > > Dmitry, why did you switch maven.compiler.target to 11 when using Java 9+ > in maven.compiler.target? Indeed it breaks compilation on Java 9-10. This > is in IGNITE-11189 > > BTW they are

Re: SpiUriDeploy TC job fails on Java 9+

2019-07-03 Thread Ilya Kasnacheev
Hello! Dmitry, why did you switch maven.compiler.target to 11 when using Java 9+ in maven.compiler.target? Indeed it breaks compilation on Java 9-10. This is in IGNITE-11189 BTW they are unsupported now, maybe we just drop them for good? Ivan what do you think? Regards, -- Ilya Kasnacheev ср,

Re: SpiUriDeploy TC job fails on Java 9+

2019-07-03 Thread Павлухин Иван
Hi Ilya, Yep there is no problem with Java 12 and 11 as well because a following option is specified in parent pom: 11 The only easy way I found is to use 9 (for java-9+ profile). Another option could be supplying maven with -Dmaven.compiler.target argument calculated for a used jdk version. Wha

Re: SpiUriDeploy TC job fails on Java 9+

2019-07-02 Thread Ilya Kasnacheev
Hello! I have just tried, looks like it is buildable with Java 12 (provided that -Dmaven.javadoc.skip=true is specified) Regards, -- Ilya Kasnacheev пн, 1 июл. 2019 г. в 22:04, Павлухин Иван : > Hi, > > SpiUriDeploy fails on TC when it is run with any Java 9+ version [1]. > The reason is very

SpiUriDeploy TC job fails on Java 9+

2019-07-01 Thread Павлухин Иван
Hi, SpiUriDeploy fails on TC when it is run with any Java 9+ version [1]. The reason is very simple, it contains one preliminary build step which is forced to run with Java 8. But for Java 9+ builds we pass module-related arguments and consequently it leads to Java 8 failure (unknown argument). I