Akmal, Nikolay, Do you have any progress on this? Let's finish with this ticket within 2.5 scope and merge it there.
-- Denis On Thu, Apr 19, 2018 at 10:07 PM, Akmal Chaudhri < akmal.chaud...@gridgain.com> wrote: > Nikolay, > > Ok. I _think_ it should be better now, so we can try again. > > 1. I removed the 3 pull requests. > 2. I renamed the Java code files with a "Java" prefix. > 3. I created a test file for the Java code, using the existing test file > for the Scala code. > 4. I updated the TestSuite with bullet #3 above. > 5. I combined all changes into a single pull request. > > Note that one of the tests still fails locally due to the path issue. I > have no idea how to fix this, although the code works fine. > > Thank you. > > > > On 19 April 2018 at 22:41, Nikolay Izhikov <nizhi...@apache.org> wrote: > > > Hello, Akmal. > > > > 1. As a first step. Let's combine all changes you want to merge into a > > single pull request. > > > > I suggest that your changes relates only to "examples" module. > > > > 2. So, when you will have one pull request, please, run "Examples" test > > suite on Team City for your branch. > > After that, attach link to execution into issue. > > > > After completing step 1 I will be able to review your changes. > > After completing step 2 and review we will be to merge your changes to > > master. > > > > В Чт, 19/04/2018 в 15:30 +0100, Akmal Chaudhri пишет: > > > Nikolay, > > > > > > The code is the same for the attached version to the ticket, as well as > > the > > > pull version. The tests are also the same as those provided for the > Scala > > > DF examples. I have checked and they work correctly with the exception > of > > > the path issue which I previously mentioned on the dev list. > > > > > > I'm afraid I am new to this whole process, so need someone in the > > community > > > to assist. > > > > > > To summarise, > > > > > > 1. The Java Spark DF code is equivalent in functionality to the Scala > > Spark > > > DF code. These Java examples were requested by Denis Magda, since only > > > Scala examples were previously available. > > > 2. The tests scripts provided with the Scala DF code work just fine > with > > > the Java code. > > > 3. There is a problem with a path issue in test #2, where the code > reads > > a > > > JSON file. Since the Java code is equivalent to the Scala code, I don't > > > know how the Scala code passed this test. > > > > > > > > > > > > On 18 April 2018 at 22:11, Nikolay Izhikov <nizhi...@apache.org> > wrote: > > > > > > > Hello, Akmal. > > > > > > > > I see 3 pull requests attached to the ticket [1], [2], [3]. > > > > I see 3 java files attached to the ticket, also. > > > > > > > > Which changes you want to be reviewed and merge? Please, clarify. > > > > > > > > Delete all unnecessary pull requests link from the ticket. > > > > Add new examples to the tests so we can test it on the Team City. > > > > You can take IgniteDataFrameSelfTest as an example. > > > > > > > > I also suggest to rename java examples with "Java" prefix. > > > > > > > > IgniteDataFrameWriteExample.java -> JavaIgniteDataFrameWriteExampl > > e.java > > > > > > > > [1] https://github.com/apache/ignite/pull/3857 > > > > [2] https://github.com/apache/ignite/pull/3858I > > > > [3] https://github.com/apache/ignite/pull/3859 > > > > [4] https://github.com/apache/ignite/blob/master/examples/ > > > > src/test/spark/org/apache/ignite/spark/examples/ > > > > IgniteDataFrameSelfTest.java > > > > > > > > > > > > > > > > > > > > В Ср, 18/04/2018 в 17:20 +0100, Akmal Chaudhri пишет: > > > > > If any community members have time, please review the code. Three > > Java > > > > > files are attached to the Jira ticket: > > > > > > > > > > https://issues.apache.org/jira/browse/IGNITE-7909 > > > > > > > > > > The code should be functionally equivalent to the Scala Data Frames > > code > > > > > that was shipped in 2.4. > > > > > > > > > > Data Frame documentation is here: > > > > > > > > > > https://apacheignite-fs.readme.io/docs/ignite-data-frame > > > > > > > > > > Thank you! > > >
