Re: Review request for IGNITE-8534 Upgrade Ignite Spark Module's Spark version to 2.3.

2018-05-30 Thread Ray
Before spark 2.3, spark is compiled using scala 2.11 and 2.10 separately. Spark-2.10 module in Ignite exists to accommodate this, it's not used anywhere else in project. Now spark 2.3 decided to remove support for scala 2.10, so we can safely remove spark-2.10 module in Ignite. It won't affect

Re: Review request for IGNITE-8534 Upgrade Ignite Spark Module's Spark version to 2.3.

2018-05-30 Thread Petr Ivanov
Ray, could you share details of your investigation regarding spark-2.10 module removal? Is it not used anywhere that it can be safely removed? And what about visor-console-2.10 and scala-2.10 modules? > On 30 May 2018, at 08:41, Ray wrote: > > Hi Dmitriy, > > Thanks for the reply. > > I

Re: Review request for IGNITE-8534 Upgrade Ignite Spark Module's Spark version to 2.3.

2018-05-29 Thread Dmitry Pavlov
Hi Ray, Could you also please resolve conflicts in PR? Conflicting files modules/spark-2.10/pom.xml вт, 29 мая 2018 г. в 18:19, Dmitry Pavlov : > Hi Ray, > > Status of this ticket is In Progress, so it is not displayed in filters. > > Could you please set status to Patch Available if PR is

Re: Review request for IGNITE-8534 Upgrade Ignite Spark Module's Spark version to 2.3.

2018-05-29 Thread Dmitry Pavlov
Hi Ray, Status of this ticket is In Progress, so it is not displayed in filters. Could you please set status to Patch Available if PR is ready for review? Sincerely, Dmitriy Pavlov пн, 28 мая 2018 г. в 8:40, Ray : > Valentin Kulichenko and Nikolay Izhikov can you please take a look at PR >

Review request for IGNITE-8534 Upgrade Ignite Spark Module's Spark version to 2.3.

2018-05-27 Thread Ray
Valentin Kulichenko and Nikolay Izhikov can you please take a look at PR and provide comments? Other reviewers are welcome as well. https://github.com/apache/ignite/pull/4033 I modified Ignite Spark module to fit Spark 2.3 APIs and removed spark-2.10 module because Spark 2.3 stooped support for