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 -> JavaIgniteDataFrameWriteExample.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! >
