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