I've added 'scala' profile to ~Build Apache Ignite~.
Can you check please that everything is OK now?

> On 5 Jun 2019, at 12:16, Nikolay Izhikov <nizhi...@apache.org> wrote:
> 
> Petr.
> 
> Current PR solves the following issue: "Data Frame examples crashes in 
> master".
> 
> We need to solve one more issue: 
> 
> "Data Frame examples not tested on TC" - this require "scala" profile in 
> "Build Ignite" suite.
> 
> 
> В Ср, 05/06/2019 в 12:05 +0300, Petr Ivanov пишет:
>>> On 5 Jun 2019, at 11:49, Nikolay Izhikov <nizhi...@apache.org> wrote:
>>> 
>>>> 1. Why does your PR show only single file changed?
>>> 
>>> Because this change fix the issue described in the ticket [1]
>> 
>> So changes in intermediate commits were reverted in branch?
>> 
>> 
>>> 
>>>> 2. Can you describe the problem once more, please? Is it about
>>> 
>>> :ignite-examples module not being build in ~Build Apache Ignite~ ?
>>> 
>>> Spark examples activated only if `scala` profile enabled.
>>> There is no `scala` profile in "~Build Apache Ignite~", only `all-scala`.
>> 
>> Let's add 'scala' profile into build.
>> 
>> 
>>> 
>>> 
>>> 
>>> [1] https://issues.apache.org/jira/browse/IGNITE-11881
>>> 
>>> ср, 5 июн. 2019 г. в 10:59, Petr Ivanov <mr.wei...@gmail.com>:
>>> 
>>>> Nikolay,
>>>> 
>>>> 
>>>> 1. Why does your PR show only single file changed?
>>>> 2. Can you describe the problem once more, please? Is it about
>>>> :ignite-examples module not being build in ~Build Apache Ignite~ ?
>>>> 
>>>> 
>>>> 
>>>>> On 4 Jun 2019, at 20:34, Nikolay Izhikov <nizhi...@apache.org> wrote:
>>>>> 
>>>>> Hello, Igniters.
>>>>> 
>>>>> Root of the issue is in our maven profiles hierarchy.
>>>>> 
>>>>> "Build Ignite" suite on TC activate 'all-scala' and examples module have
>>>> 
>>>> only 'scala' profile.
>>>>> To reduce copy-pase I added 'scala' profile of examples module
>>>> 
>>>> activation based on system property 'scala-examples'.
>>>>> 
>>>>> So, for now my PR [1] fixes both issues with examples module:
>>>>> 
>>>>> 1. Unnecessary dependecies that leads to Spark examples crash.
>>>>> 2. Spark examples not tested on TC.
>>>>> 
>>>>> After merge of PR [1] we should add this property to "Build Ignite"
>>>> 
>>>> suite.
>>>>> 
>>>>> Is there any objections?
>>>>> 
>>>>> [1] https://github.com/apache/ignite/pull/6587
>>>>> 
>>>>> [2]
>>>> 
>>>> https://ci.ignite.apache.org/viewLog.html?buildId=4041670&buildTypeId=IgniteTests24Java8_Examples&tab=testsInfo&branch_IgniteTests24Java8=pull%2F6587%2Fhead
>>>>> 
>>>>> 
>>>>> В Вт, 04/06/2019 в 16:56 +0300, Nikolay Izhikov пишет:
>>>>>> I moved jackson dependencies to maven-javadoc-plugin additional
>>>> 
>>>> dependencies.
>>>>>> Seems, it solves both issues: Data Frame examples and javadoc
>>>> 
>>>> generation.
>>>>>> 
>>>>>> But, Data Frame examples still doesn't shows in tc log.
>>>>>> 
>>>>>> Will investigate it, shortly.
>>>>>> 
>>>>>> 
>>>> 
>>>> https://ci.ignite.apache.org/viewLog.html?buildId=4039945&tab=queuedBuildOverviewTab
>>>>>> 
>>>>>> https://ci.ignite.apache.org/viewLog.html?buildId=4039943&;;;;
>>>>>> 
>>>>>> В Вт, 04/06/2019 в 15:45 +0300, Павлухин Иван пишет:
>>>>>>> Would be great if it is possible.
>>>>>>> 
>>>>>>> вт, 4 июн. 2019 г. в 15:43, Nikolay Izhikov <nizhi...@apache.org>:
>>>>>>>> 
>>>>>>>> I used to think it's forbidden to use direct version in pom.xml.
>>>>>>>> We should keep all versions in parent pom.
>>>>>>>> 
>>>>>>>> 
>>>>>>>> В Вт, 04/06/2019 в 15:36 +0300, Павлухин Иван пишет:
>>>>>>>>> Nikolay,
>>>>>>>>> 
>>>>>>>>> It was already observed that removal of jackson dependencies from
>>>>>>>>> examples module would lead to a Javadoc build job failure (for
>>>>>>>>> examples module). One way to fix it is changing jackson dependency
>>>>>>>>> versions:
>>>>>>>>> <dependency>
>>>>>>>>>  <groupId>com.fasterxml.jackson.core</groupId>
>>>>>>>>>  <artifactId>jackson-core</artifactId>
>>>>>>>>>  <version>2.6.7</version>
>>>>>>>>> </dependency>
>>>>>>>>> 
>>>>>>>>> <dependency>
>>>>>>>>>  <groupId>com.fasterxml.jackson.core</groupId>
>>>>>>>>>  <artifactId>jackson-databind</artifactId>
>>>>>>>>>  <version>2.6.7</version>
>>>>>>>>> </dependency>
>>>>>>>>> 
>>>>>>>>> <dependency>
>>>>>>>>>  <groupId>com.fasterxml.jackson.core</groupId>
>>>>>>>>>  <artifactId>jackson-annotations</artifactId>
>>>>>>>>>  <version>2.6.7</version>
>>>>>>>>> </dependency>
>>>>>>>>> 
>>>>>>>>> вт, 4 июн. 2019 г. в 14:57, Nikolay Izhikov <nizhi...@apache.org>:
>>>>>>>>>> 
>>>>>>>>>> Igniters
>>>>>>>>>> 
>>>>>>>>>> I've found the root issue.
>>>>>>>>>> 
>>>>>>>>>> jackson dependency was added in IGNITE-10803 and that's brokes
>>>> 
>>>> Spark Examples.
>>>>>>>>>> 
>>>>>>>>>> Alex, Yury, please, clarify, do we really need this dependencies?
>>>>>>>>>> 
>>>>>>>>>> В Вт, 04/06/2019 в 11:42 +0300, Павлухин Иван пишет:
>>>>>>>>>>> But -Pscala is already activated. But it seems that a build job in
>>>>>>>>>>> question does not compile classes and retrieves a compiled package
>>>>>>>>>>> from ~Build Apache Ignite~ job. Need to check if spark examples are
>>>>>>>>>>> present.
>>>>>>>>>>> 
>>>>>>>>>>> вт, 4 июн. 2019 г. в 09:52, Павлухин Иван <vololo...@gmail.com>:
>>>>>>>>>>>> 
>>>>>>>>>>>> Nikolay,
>>>>>>>>>>>> 
>>>>>>>>>>>> It looks like that profile -Pscala should be activated to run
>>>>>>>>>>>> IgniteExamplesSparkSelfTestSuite. I was able to start that suite
>>>>>>>>>>>> execution on my machine but it failed during execution.
>>>>>>>>>>>> 
>>>>>>>>>>>> вт, 4 июн. 2019 г. в 03:08, Denis Magda <dma...@apache.org>:
>>>>>>>>>>>>> 
>>>>>>>>>>>>> Hey Peter, Anton,
>>>>>>>>>>>>> 
>>>>>>>>>>>>> Are any of you able to help Nickolay with [1]?
>>>>>>>>>>>>> 
>>>>>>>>>>>>> -
>>>>>>>>>>>>> Denis
>>>>>>>>>>>>> 
>>>>>>>>>>>>> 
>>>>>>>>>>>>> On Mon, Jun 3, 2019 at 6:17 AM Nikolay Izhikov <
>>>> 
>>>> nizhi...@apache.org> wrote:
>>>>>>>>>>>>> 
>>>>>>>>>>>>>> Hello, Igniters.
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> We have several issues with Examples suite on TC
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> 1. IgniteExamplesSparkSelfTestSuite seems configure, bu not
>>>> 
>>>> runs on suite
>>>>>>>>>>>>>> execution. [1]
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> 2. Spark Data Frame examples doesn't work in master. Seems some
>>>> 
>>>> classpath
>>>>>>>>>>>>>> issues [2]
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> I will resolve issue #2.
>>>>>>>>>>>>>> Can anyone help me with #1?
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> [1]
>>>>>>>>>>>>>> 
>>>> 
>>>> https://ci.ignite.apache.org/viewLog.html?buildId=4027327&buildTypeId=IgniteTests24Java8_Examples&tab=buildParameters
>>>>>>>>>>>>>> [2] https://issues.apache.org/jira/browse/IGNITE-11881
>>>>>>>>>>>>>> 
>>>>>>>>>>>> 
>>>>>>>>>>>> 
>>>>>>>>>>>> 
>>>>>>>>>>>> --
>>>>>>>>>>>> Best regards,
>>>>>>>>>>>> Ivan Pavlukhin
>>>>>>>>>>> 
>>>>>>>>>>> 
>>>>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> 
>>>>>>> 
>>>>>>> 
>>>>>>> 
>>>> 
>>>> 
>> 
>> 

Reply via email to