Hi all. I wanted to inform the tasks I worked on this week: Feb 13 - Feb 19 .
* Develop method newQuery. * Develop classes Neo4jResult and Neo4jQuery. * Develop method execute(query). * Activate tests: testTruncateSchema testDeleteSchema testQuery testQueryStartKey testQueryEndKey testQueryKeyRange testQueryWebPageSingleKey testQueryWebPageSingleKeyDefaultFields testQueryWebPageQueryEmptyResults testDelete testGetPartitions testResultSize testResultSizeStartKey testResultSizeEndKey testResultSizeKeyRange testResultSizeWithLimit testResultSizeStartKeyWithLimit testResultSizeEndKeyWithLimit testResultSizeKeyRangeWithLimit Also, I have a question, When the method Neo4jStore#delete(key) should return true or false?, I found out that I had to always return true in order to pass the tests, is that correct? My code is here: https://github.com/mgov88/gora/tree/GORA-663 Regards, Gaby El lun, 15 de feb. de 2021 a la(s) 19:01, John Mora (jhnmora...@gmail.com) escribió: > Hi Gaby > > Thanks for the update. > > Overall the code looks good, I do not have specific feedback for you this > week. > > According to your proposed timeline you should start working on the Query > features, let's do it. Let me know if you have questions. > > > Thanks, > John > > El sáb, 13 feb 2021 a las 0:57, gabriela ortiz (<arqgabyor...@gmail.com>) > escribió: > >> Hi all. >> >> I wanted to inform the tasks I worked on this week: Feb 06 - Feb 12 . >> >> * Enhance variable names. >> * Add enum for neo4j protocols. >> * Enhance getUnionSchema method for Maps. >> * Implement partitons. >> * Activate tests: >> testUpdate >> testGetRecursive >> testGetDoubleRecursive >> testGetWebPage >> testGetWebPageDefaultFields >> >> Also, I started working on my C.V. >> >> My code is here: https://github.com/mgov88/gora/tree/GORA-663 >> >> Regards, >> Gaby >> >> El mié, 10 de feb. de 2021 a la(s) 21:33, gabriela ortiz ( >> arqgabyor...@gmail.com) escribió: >> >>> Hi John. >>> >>> Thanks for the feedback I will work on your comments. >>> >>> Regards, >>> Gaby >>> >>> >>> El mié, 10 de feb. de 2021 a la(s) 12:04, John Mora ( >>> jhnmora...@gmail.com) escribió: >>> >>>> Hi Gaby >>>> >>>> Thanks for the update. >>>> >>>> BTW, I am sorry that I did not provide feedback on your code last week, >>>> I have been busy. >>>> >>>> Some comments: >>>> >>>> Please use more descriptive variable names: >>>> >>>> >>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L368 >>>> >>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L165 >>>> >>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L171 >>>> >>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L193 >>>> >>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L194 >>>> >>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L200 >>>> >>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L206 >>>> >>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L216 >>>> >>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L216 >>>> >>>> Typo: >>>> >>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L216 >>>> >>>> Avoid string concatenation: >>>> >>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L307 >>>> >>>> Use an Enum instead of string literals, >>>> >>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L129 >>>> >>>> >>>> regards >>>> John >>>> >>>> El lun, 8 feb 2021 a las 2:08, gabriela ortiz (<arqgabyor...@gmail.com>) >>>> escribió: >>>> >>>>> Hi all. >>>>> >>>>> I wanted to inform the tasks I worked on this week: Jan 30 - Feb 05 . >>>>> >>>>> * Enhance the deleteSchema method (delete existing nodes when deleting >>>>> the schema constraints) >>>>> * Enhance Map, Record, Array and Bytes serialization / >>>>> de-serialization process using Base64 encoding. >>>>> * Activate tests: >>>>> testPutNested >>>>> testPutArray >>>>> testPutBytes >>>>> testPutMap >>>>> testPutMixedMaps >>>>> testGetNested >>>>> testGet3UnionField >>>>> testGetWithFields >>>>> >>>>> My code is here: https://github.com/mgov88/gora/tree/GORA-663 >>>>> >>>>> Regards, >>>>> Gaby >>>>> >>>>> El lun, 1 de feb. de 2021 a la(s) 01:53, gabriela ortiz ( >>>>> arqgabyor...@gmail.com) escribió: >>>>> >>>>>> Hi all. >>>>>> >>>>>> I wanted to inform the tasks I worked on this week: Jan 17 - Jan 29 . >>>>>> >>>>>> * Add suggested javadocs. >>>>>> * Add suggested constants. >>>>>> * Make EXIST constraints optional. >>>>>> * Activate tests: >>>>>> testNewInstance >>>>>> testAutoCreateSchema >>>>>> testPut >>>>>> testBenchmarkExists >>>>>> testGetNonExisting >>>>>> testObjectFieldValue >>>>>> * Write a blog (Career opportunities) >>>>>> >>>>>> My code is here: https://github.com/mgov88/gora/tree/GORA-663 >>>>>> >>>>>> Regards, >>>>>> Gaby >>>>>> >>>>>> >>>>>> El mié, 27 de ene. de 2021 a la(s) 12:00, John Mora ( >>>>>> jhnmora...@gmail.com) escribió: >>>>>> >>>>>>> Hi Gaby >>>>>>> >>>>>>> Thanks for your report. >>>>>>> >>>>>>> Some comments: >>>>>>> >>>>>>> Please use constants instead of literal values here: >>>>>>> >>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L204 >>>>>>> >>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L224 >>>>>>> >>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L251 >>>>>>> >>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L341 >>>>>>> >>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L284 >>>>>>> >>>>>>> Add javadocs here: >>>>>>> >>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L278 >>>>>>> >>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L388 >>>>>>> >>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L431 >>>>>>> >>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L487 >>>>>>> >>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L500 >>>>>>> >>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L507 >>>>>>> >>>>>>> Regarding the EXISTS constraint: IMHO that is a good approach, but >>>>>>> let's make this attribute optional in the mapping and default: false. >>>>>>> >>>>>>> Regards, >>>>>>> John >>>>>>> >>>>>>> El lun, 25 ene 2021 a las 18:32, gabriela ortiz (< >>>>>>> arqgabyor...@gmail.com>) escribió: >>>>>>> >>>>>>>> Hi all. >>>>>>>> >>>>>>>> I wanted to inform the tasks I worked on this week: Jan 16 - Jan 22 >>>>>>>> . >>>>>>>> >>>>>>>> - Add EXISTS constraints for properties. >>>>>>>> - Develop put method. >>>>>>>> - Enable test of the put & get methods. >>>>>>>> - Switch to fixed version docker image 'neo4j:4.2.2-enterprise' >>>>>>>> instead of 'enterprise' tag. >>>>>>>> >>>>>>>> My code is here: https://github.com/mgov88/gora/tree/GORA-663 >>>>>>>> >>>>>>>> I have a question, in Neo4j the null value represent the absence of >>>>>>>> the property/relation: >>>>>>>> >>>>>>>> "In Neo4j, since there is no table schema or equivalent to restrict >>>>>>>> possible properties, non-existence and null are equivalent for node and >>>>>>>> relationship properties. That is, there really is no such thing as a >>>>>>>> property with a null value; null indicates that the property doesn’t >>>>>>>> exist >>>>>>>> at all." >>>>>>>> >>>>>>>> >>>>>>>> https://neo4j.com/developer/kb/understanding-non-existent-properties-and-null-values/ >>>>>>>> >>>>>>>> I added an attribute in the mapping file to specify if the EXISTS >>>>>>>> constraint should be applied, because in the tests sometimes the field >>>>>>>> values are null and there are conflicts with this constraint. >>>>>>>> >>>>>>>> >>>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/test/resources/gora-neo4j-mapping.xml#L22 >>>>>>>> >>>>>>>> Is it correct this way? >>>>>>>> >>>>>>>> Regardss >>>>>>>> Gaby >>>>>>>> >>>>>>>> El lun, 18 de ene. de 2021 a la(s) 22:58, gabriela ortiz ( >>>>>>>> arqgabyor...@gmail.com) escribió: >>>>>>>> >>>>>>>>> Hi >>>>>>>>> >>>>>>>>> Thanks for your comments. I am working on them. >>>>>>>>> >>>>>>>>> Best regards, >>>>>>>>> Gaby >>>>>>>>> >>>>>>>>> El lun, 18 de ene. de 2021 a la(s) 17:11, John Mora ( >>>>>>>>> jhnmora...@gmail.com) escribió: >>>>>>>>> >>>>>>>>>> Hi Gaby. >>>>>>>>>> >>>>>>>>>> Thanks for your update. >>>>>>>>>> >>>>>>>>>> Some comments. >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L156 >>>>>>>>>> >>>>>>>>>> Please create EXISTS constraints for the other properties of the >>>>>>>>>> Node as well. >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> https://neo4j.com/docs/cypher-manual/current/administration/constraints/#administration-constraints-prop-exist-nodes >>>>>>>>>> >>>>>>>>>> You can use the mapping parsed from the XML. >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/mapping/Neo4jMapping.java#L38 >>>>>>>>>> >>>>>>>>>> Please use a specific docker image tag of Neo4j instead of the >>>>>>>>>> 'latest' or 'enterprise' >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/test/java/org/apache/gora/neo4j/GoraNeo4jTestDriver.java#L60 >>>>>>>>>> >>>>>>>>>> This might break the code when a new version of Neo4j is released. >>>>>>>>>> >>>>>>>>>> Regards, >>>>>>>>>> John >>>>>>>>>> >>>>>>>>>> El dom, 17 ene 2021 a las 14:39, gabriela ortiz (< >>>>>>>>>> arqgabyor...@gmail.com>) escribió: >>>>>>>>>> >>>>>>>>>>> Hi all. >>>>>>>>>>> >>>>>>>>>>> I wanted to inform the tasks I worked on this week: Jan 09 - Jan >>>>>>>>>>> 15 . >>>>>>>>>>> >>>>>>>>>>> - Enhance the javadocs documentation. >>>>>>>>>>> - Replace literal values with constants. >>>>>>>>>>> - Develop the put() method of the datastore. >>>>>>>>>>> - Test the put / exists methods. >>>>>>>>>>> - Write a blog for this week. >>>>>>>>>>> >>>>>>>>>>> My code is here: https://github.com/mgov88/gora/tree/GORA-663 >>>>>>>>>>> >>>>>>>>>>> Regardss >>>>>>>>>>> Gaby >>>>>>>>>>> >>>>>>>>>>> El lun, 11 de ene. de 2021 a la(s) 09:46, gabriela ortiz ( >>>>>>>>>>> arqgabyor...@gmail.com) escribió: >>>>>>>>>>> >>>>>>>>>>>> Hi >>>>>>>>>>>> >>>>>>>>>>>> Thanks for your comments. I will add the missing documentation >>>>>>>>>>>> and thanks for your reminder I will fill the form now. >>>>>>>>>>>> >>>>>>>>>>>> Best regards, >>>>>>>>>>>> Gaby >>>>>>>>>>>> >>>>>>>>>>>> El lun, 11 de ene. de 2021 a la(s) 00:16, John Mora ( >>>>>>>>>>>> jhnmora...@gmail.com) escribió: >>>>>>>>>>>> >>>>>>>>>>>>> Hi. >>>>>>>>>>>>> >>>>>>>>>>>>> Thanks for the update. >>>>>>>>>>>>> >>>>>>>>>>>>> Some comments: >>>>>>>>>>>>> >>>>>>>>>>>>> Please use constants instead of literal values here: >>>>>>>>>>>>> >>>>>>>>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/test/java/org/apache/gora/neo4j/GoraNeo4jTestDriver.java#L37 >>>>>>>>>>>>> >>>>>>>>>>>>> Add javadocs in these classes: >>>>>>>>>>>>> >>>>>>>>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/test/java/org/apache/gora/neo4j/store/Neo4jStoreTest.java >>>>>>>>>>>>> >>>>>>>>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/test/java/org/apache/gora/neo4j/GoraNeo4jTestDriver.java >>>>>>>>>>>>> >>>>>>>>>>>>> Add descriptions for the attributes of these classes: >>>>>>>>>>>>> >>>>>>>>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/mapping/Neo4jMapping.java >>>>>>>>>>>>> >>>>>>>>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/mapping/Property.java >>>>>>>>>>>>> >>>>>>>>>>>>> Please do not forget to submit your midpoint feedback. It is >>>>>>>>>>>>> due Jan. 12 4pm UTC. >>>>>>>>>>>>> >>>>>>>>>>>>> Cheers, >>>>>>>>>>>>> John >>>>>>>>>>>>> >>>>>>>>>>>>> El sáb, 9 ene 2021 a las 18:18, gabriela ortiz (< >>>>>>>>>>>>> arqgabyor...@gmail.com>) escribió: >>>>>>>>>>>>> >>>>>>>>>>>>>> Hi all. >>>>>>>>>>>>>> >>>>>>>>>>>>>> I wanted to inform the tasks I worked on this week: Jan 01 - >>>>>>>>>>>>>> Jan 08 >>>>>>>>>>>>>> >>>>>>>>>>>>>> - Develop a test for XSD validations. >>>>>>>>>>>>>> - Develop methods: exists and delete in the datastore. >>>>>>>>>>>>>> - Switch to Neo4j Enterprise from Neo4j Community in the >>>>>>>>>>>>>> container of the tests. >>>>>>>>>>>>>> - Add comments and javadocs >>>>>>>>>>>>>> - Change the UNIQUE indexes for NODE KEY indexes (present in >>>>>>>>>>>>>> the Neo4j Enterprise version) >>>>>>>>>>>>>> >>>>>>>>>>>>>> My code is here: https://github.com/mgov88/gora/tree/GORA-663 >>>>>>>>>>>>>> >>>>>>>>>>>>>> Regardss >>>>>>>>>>>>>> Gaby >>>>>>>>>>>>>> >>>>>>>>>>>>>> El jue, 7 de ene. de 2021 a la(s) 13:28, gabriela ortiz ( >>>>>>>>>>>>>> arqgabyor...@gmail.com) escribió: >>>>>>>>>>>>>> >>>>>>>>>>>>>>> Hi John >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Thanks for your comments. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> El mié, 6 de ene. de 2021 a la(s) 16:10, John Mora ( >>>>>>>>>>>>>>> jhnmora...@gmail.com) escribió: >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Hi >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Thanks for your report. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Some comments: >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> You are creating a UNIQUE index instead of the NODE KEY you >>>>>>>>>>>>>>>> mentioned on your design document. why? >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jStore.java#L141 >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>> I found this limitation in the Neo4j community version: >>>>>>>>>>>>>>> "Node key constraints, node property existence constraints and >>>>>>>>>>>>>>> relationship >>>>>>>>>>>>>>> property existence constraints are only available in Neo4j >>>>>>>>>>>>>>> Enterprise >>>>>>>>>>>>>>> Edition". >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> And, the UNIQUE indexes work in the community version. >>>>>>>>>>>>>>> Should I seek a trial version of the enterprise version or work >>>>>>>>>>>>>>> only with >>>>>>>>>>>>>>> the UNIQUE indexes of the community version? >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Also, can you write the CREATE CONSTRAINT query with >>>>>>>>>>>>>>>> Cypher-DSL? >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>> I could not find a API for constraints creation in Cypher >>>>>>>>>>>>>>> DSL. The create statements that I found are only for nodes. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Please add javadoc documentation and comments on these >>>>>>>>>>>>>>>> classes: >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jConstants.java >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/java/org/apache/gora/neo4j/store/Neo4jParameters.java >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> I will add the documentation >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Please add a test for the XSD Validation >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> https://github.com/mgov88/gora/blob/GORA-663/gora-neo4j/src/main/resources/gora-neo4j.xsd >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> I will add a test for this. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Cheers, >>>>>>>>>>>>>>>> John >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> El dom, 3 ene 2021 a las 23:12, gabriela ortiz (< >>>>>>>>>>>>>>>> arqgabyor...@gmail.com>) escribió: >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> Hi >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> I wanted to inform the tasks I worked on this week: Dec 26 >>>>>>>>>>>>>>>>> - Dec 31 >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> * Develop the Datastore methods: getSchemaName, createSchema, >>>>>>>>>>>>>>>>> deleteSchema, schemaExists >>>>>>>>>>>>>>>>> * Enable tests in Neo4jStoreTest : testCreateSchema , >>>>>>>>>>>>>>>>> testSchemaExists >>>>>>>>>>>>>>>>> * Enhance the implementation of the connection to the >>>>>>>>>>>>>>>>> backend. >>>>>>>>>>>>>>>>> * Write a blog about my project. >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> You can find my code in my branch: >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> https://github.com/mgov88/gora/tree/GORA-663 >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> Best Regards, >>>>>>>>>>>>>>>>> Gaby >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> El mié, 30 de dic. de 2020 a la(s) 13:28, John Mora ( >>>>>>>>>>>>>>>>> jhnmora...@gmail.com) escribió: >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> Hi >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> Thanks for your report. >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> The Utils package is specific for each datastore; it is >>>>>>>>>>>>>>>>>> not mandatory. You can move some code there to improve the >>>>>>>>>>>>>>>>>> readability of >>>>>>>>>>>>>>>>>> the core classes of the datastore but it is up to you. >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> I could not find your last changes in your branch in >>>>>>>>>>>>>>>>>> github, please make sure to push the commits, so I can >>>>>>>>>>>>>>>>>> review the code. >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> Cheers, >>>>>>>>>>>>>>>>>> John >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> El mar, 29 dic 2020 a las 1:03, gabriela ortiz (< >>>>>>>>>>>>>>>>>> arqgabyor...@gmail.com>) escribió: >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> Hi >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> I wanted to inform the tasks I worked on this week: Dec >>>>>>>>>>>>>>>>>>> 19 - Dec 25 >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> * I developed a GoraTestDriver for Neo4j using >>>>>>>>>>>>>>>>>>> GenericContainer >>>>>>>>>>>>>>>>>>> from the Test Containers library >>>>>>>>>>>>>>>>>>> * I developed the initialize method from Neo4jStore to >>>>>>>>>>>>>>>>>>> establish connection to the backend (Neo4j) >>>>>>>>>>>>>>>>>>> * I developed a XSD validation file for the XML mapping >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> I have a question, I realized that some Datastores have >>>>>>>>>>>>>>>>>>> 'utils' packages in the code, are they obligatory? What >>>>>>>>>>>>>>>>>>> kind of code should >>>>>>>>>>>>>>>>>>> go in them? >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> Best Regards, >>>>>>>>>>>>>>>>>>> Gaby >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> El lun, 21 de dic. de 2020 a la(s) 15:45, gabriela ortiz >>>>>>>>>>>>>>>>>>> (arqgabyor...@gmail.com) escribió: >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> Hi John >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> El sáb, 19 de dic. de 2020 a la(s) 17:13, John Mora ( >>>>>>>>>>>>>>>>>>>> jhnmora...@gmail.com) escribió: >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> Hi Gaby >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> Thanks for your report and your hard work! >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> I left some comments on your design document please >>>>>>>>>>>>>>>>>>>>> give them a look. >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> Thanks you for your comments, >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> Some comments about your code: >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> IMHO, the attributes unique, exists and index should >>>>>>>>>>>>>>>>>>>>> be optional (default: false) [0]. >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> I agree I will change that >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> The KeyClassCanonicalName and PersistentClassCanonicalName >>>>>>>>>>>>>>>>>>>>> should be read from the datastore not hard coded [1] >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> I put that hard coded because I had an error because >>>>>>>>>>>>>>>>>>>> the Datastore methods are not yet implemented, but I will >>>>>>>>>>>>>>>>>>>> change that. >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> I think this class [2] should be deleted, there are >>>>>>>>>>>>>>>>>>>>> proper tests for this in the DataStoreTestBase class. If >>>>>>>>>>>>>>>>>>>>> you are using this >>>>>>>>>>>>>>>>>>>>> for your own experiments please do not push it to github. >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> yes, I am using this class to temporary test my code, >>>>>>>>>>>>>>>>>>>> you are right I will remove it from github. >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> Thanks you for your feedback. >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> Regards, >>>>>>>>>>>>>>>>>>>> Gaby >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> [0] >>>>>>>>>>>>>>>>>>>>> https://github.com/mgov88/gora/blob/867d06e1fcf29e90b0431b9a42b82810c5409dda/gora-neo4j/src/test/resources/gora-neo4j-mapping.xml >>>>>>>>>>>>>>>>>>>>> [1] >>>>>>>>>>>>>>>>>>>>> https://github.com/mgov88/gora/blob/867d06e1fcf29e90b0431b9a42b82810c5409dda/gora-neo4j/src/main/java/org/apache/gora/neo4j/mapping/Neo4jMappingBuilder.java#L44 >>>>>>>>>>>>>>>>>>>>> [2] >>>>>>>>>>>>>>>>>>>>> https://github.com/mgov88/gora/blob/867d06e1fcf29e90b0431b9a42b82810c5409dda/gora-neo4j/src/main/java/org/apache/gora/neo4j/experimental/experiment.java >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> Cheers >>>>>>>>>>>>>>>>>>>>> John >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> El vie, 18 dic 2020 a las 23:06, gabriela ortiz (< >>>>>>>>>>>>>>>>>>>>> arqgabyor...@gmail.com>) escribió: >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>> Hi >>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>> I wanted to inform the tasks I worked on this week: >>>>>>>>>>>>>>>>>>>>>> Dec 12 - Dec 18 >>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>> * I wrote a blog about the struggles learning CQL of >>>>>>>>>>>>>>>>>>>>>> Neo4j : >>>>>>>>>>>>>>>>>>>>>> https://gabycoder88.wordpress.com/2020/12/18/discovering-cypher-everybody-struggles/ >>>>>>>>>>>>>>>>>>>>>> * I created the suggested design document (link below >>>>>>>>>>>>>>>>>>>>>> ) with details about the Neo4j datastore. >>>>>>>>>>>>>>>>>>>>>> * I finished creating the structure of the datastore >>>>>>>>>>>>>>>>>>>>>> project. >>>>>>>>>>>>>>>>>>>>>> * I developed and tested the parsing of XML mappings >>>>>>>>>>>>>>>>>>>>>> for the datastore from gora-neo4j-mapping.xml >>>>>>>>>>>>>>>>>>>>>> * I developed and tested the read of connection >>>>>>>>>>>>>>>>>>>>>> properties from gora.properties >>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>> My code is here: >>>>>>>>>>>>>>>>>>>>>> https://github.com/mgov88/gora/commits/GORA-663 >>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>> My design document: >>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>> https://docs.google.com/document/d/1fQbCHNIFEKxAUlOuwqDRDdnpecLk2S4TkOJSWIk6qCE/edit?usp=sharing >>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>> Best Regards, >>>>>>>>>>>>>>>>>>>>>> Gaby >>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>> El lun, 14 de dic. de 2020 a la(s) 15:25, gabriela >>>>>>>>>>>>>>>>>>>>>> ortiz (arqgabyor...@gmail.com) escribió: >>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> Hi John. >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> Thanks very much for your feedback. I shall work on >>>>>>>>>>>>>>>>>>>>>>> the document and the structure of the datastore project. >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> Best regards, >>>>>>>>>>>>>>>>>>>>>>> Gaby >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> El sáb, 12 de dic. de 2020 a la(s) 17:43, John Mora ( >>>>>>>>>>>>>>>>>>>>>>> jhnmora...@gmail.com) escribió: >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>> Hi Gaby, >>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>> Thanks for the update and your hard work. >>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>> Do not worry about the paid versions of Neo4j. >>>>>>>>>>>>>>>>>>>>>>>> They all share a common interface (Cypher Graph Query >>>>>>>>>>>>>>>>>>>>>>>> Language) and connect >>>>>>>>>>>>>>>>>>>>>>>> using the same protocols, so it is fine that your work >>>>>>>>>>>>>>>>>>>>>>>> only with community >>>>>>>>>>>>>>>>>>>>>>>> version. >>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>> Nice to see that you added a basic input/output >>>>>>>>>>>>>>>>>>>>>>>> example working with Neo4j but eventually you should >>>>>>>>>>>>>>>>>>>>>>>> move it to another >>>>>>>>>>>>>>>>>>>>>>>> project. Now let's create all the packages and >>>>>>>>>>>>>>>>>>>>>>>> resources needed for the >>>>>>>>>>>>>>>>>>>>>>>> structure of a datastore. >>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>> I think you should put your ideas in a Google Docs >>>>>>>>>>>>>>>>>>>>>>>> document and share in the mailing list. >>>>>>>>>>>>>>>>>>>>>>>> * Your UML design* The branch that you are working >>>>>>>>>>>>>>>>>>>>>>>> on.* How the connection will be implemented.* Which >>>>>>>>>>>>>>>>>>>>>>>> connection parameters >>>>>>>>>>>>>>>>>>>>>>>> the datastore will need,* How you will set up a >>>>>>>>>>>>>>>>>>>>>>>> testing instance in the >>>>>>>>>>>>>>>>>>>>>>>> unit test of Gora.* The data type mapping between your >>>>>>>>>>>>>>>>>>>>>>>> datastore and Java >>>>>>>>>>>>>>>>>>>>>>>> data types. >>>>>>>>>>>>>>>>>>>>>>>> Keep it simple, here [0] you have a document that I >>>>>>>>>>>>>>>>>>>>>>>> made for the Kudu datastore a couple of years ago. >>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>> [0] >>>>>>>>>>>>>>>>>>>>>>>> https://docs.google.com/document/d/1colS1ooQZIvuJcnx6DSsZIgesokK8TjwaEfgdygt4mo/edit#heading=h.pcphnswc4uen >>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>> Best regards, >>>>>>>>>>>>>>>>>>>>>>>> John >>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>> El vie, 11 dic 2020 a las 23:06, gabriela ortiz (< >>>>>>>>>>>>>>>>>>>>>>>> arqgabyor...@gmail.com>) escribió: >>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>> Hi all. >>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>> I wanted to inform the tasks I worked on this >>>>>>>>>>>>>>>>>>>>>>>>> week: Dec 06 - Dec 11 >>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>> - I created a new branch for my project : >>>>>>>>>>>>>>>>>>>>>>>>> https://github.com/mgov88/gora/tree/GORA-663 >>>>>>>>>>>>>>>>>>>>>>>>> - I created a new module for datastore gora-neo4j >>>>>>>>>>>>>>>>>>>>>>>>> - I add the maven dependencies for neo4j JDBC and >>>>>>>>>>>>>>>>>>>>>>>>> Cypher-DSL >>>>>>>>>>>>>>>>>>>>>>>>> - I read the JDBC and DSL documentation. >>>>>>>>>>>>>>>>>>>>>>>>> - I implemented a example inserting and querying >>>>>>>>>>>>>>>>>>>>>>>>> nodes into neo4j >>>>>>>>>>>>>>>>>>>>>>>>> - I joined the Interns chat session of this week. >>>>>>>>>>>>>>>>>>>>>>>>> - I submitted the forms asked by Outreachy. >>>>>>>>>>>>>>>>>>>>>>>>> - I submitted the initial feedback of Outreachy. >>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>> I have a question, neo4j has many editions ( >>>>>>>>>>>>>>>>>>>>>>>>> https://neo4j.com/subscriptions/ ) and the docker >>>>>>>>>>>>>>>>>>>>>>>>> that I have been using is the community. >>>>>>>>>>>>>>>>>>>>>>>>> Should I be test all the editions, there are demos >>>>>>>>>>>>>>>>>>>>>>>>> or open server that I can use instead of buying a >>>>>>>>>>>>>>>>>>>>>>>>> subscription ? >>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>> Regardss >>>>>>>>>>>>>>>>>>>>>>>>> Gaby >>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>> El sáb, 5 de dic. de 2020 a la(s) 15:21, John Mora >>>>>>>>>>>>>>>>>>>>>>>>> (jhnmora...@gmail.com) escribió: >>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>> Hi Gaby. >>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>> Thanks for your work and the update of your >>>>>>>>>>>>>>>>>>>>>>>>>> progress. >>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>> Working with docker containers is totally fine, >>>>>>>>>>>>>>>>>>>>>>>>>> but take into account that you need to expose all >>>>>>>>>>>>>>>>>>>>>>>>>> the necessary ports in >>>>>>>>>>>>>>>>>>>>>>>>>> the container for the communication with Apache >>>>>>>>>>>>>>>>>>>>>>>>>> Gora. Also you can start >>>>>>>>>>>>>>>>>>>>>>>>>> checking out testcontainers [1] for the unit test of >>>>>>>>>>>>>>>>>>>>>>>>>> the new datastore. >>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>> Your design looks good to me, I noticed that you >>>>>>>>>>>>>>>>>>>>>>>>>> plan to use the Neo4j driver. If you have some >>>>>>>>>>>>>>>>>>>>>>>>>> experience working with JDBC >>>>>>>>>>>>>>>>>>>>>>>>>> you can also use the Neo4j JDBC Driver [2]. But it >>>>>>>>>>>>>>>>>>>>>>>>>> is up to you, it should >>>>>>>>>>>>>>>>>>>>>>>>>> not have any impact on the functionality of the >>>>>>>>>>>>>>>>>>>>>>>>>> datastore. >>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>> [1] https://www.testcontainers.org/ >>>>>>>>>>>>>>>>>>>>>>>>>> [2] https://github.com/neo4j-contrib/neo4j-jdbc >>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>> Regards, >>>>>>>>>>>>>>>>>>>>>>>>>> John >>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>> El sáb, 5 dic 2020 a las 8:20, Madhawa Kasun >>>>>>>>>>>>>>>>>>>>>>>>>> Gunasekara (<madhaw...@gmail.com>) escribió: >>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>> Hi Gaby, >>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>> Thanks for the update. Yes, we haven't tested >>>>>>>>>>>>>>>>>>>>>>>>>>> Gora in java11, So I don't think you should spend >>>>>>>>>>>>>>>>>>>>>>>>>>> time on installing neo4j >>>>>>>>>>>>>>>>>>>>>>>>>>> locally. docker container is enough. >>>>>>>>>>>>>>>>>>>>>>>>>>> Please add your project proposal to the >>>>>>>>>>>>>>>>>>>>>>>>>>> confluence page as well [1]. I wish you good luck >>>>>>>>>>>>>>>>>>>>>>>>>>> with this project. Let me >>>>>>>>>>>>>>>>>>>>>>>>>>> know if you have any questions. >>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>> [1] >>>>>>>>>>>>>>>>>>>>>>>>>>> https://cwiki.apache.org/confluence/display/GORA/Outreachy%3A+2020+Winter >>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>> Thanks, >>>>>>>>>>>>>>>>>>>>>>>>>>> Madhawa >>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>> On Sat, Dec 5, 2020 at 6:29 AM gabriela ortiz < >>>>>>>>>>>>>>>>>>>>>>>>>>> arqgabyor...@gmail.com> wrote: >>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>> Hi >>>>>>>>>>>>>>>>>>>>>>>>>>>> I wanted to inform the tasks I worked on this >>>>>>>>>>>>>>>>>>>>>>>>>>>> week: Dec 01 - Dec 05 >>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>> 1. I created a personal Blog for Outreachy >>>>>>>>>>>>>>>>>>>>>>>>>>>> reports: https://gabycoder88.wordpress.com/ >>>>>>>>>>>>>>>>>>>>>>>>>>>> 2. I wrote the first post of my blog titled >>>>>>>>>>>>>>>>>>>>>>>>>>>> 'About me': >>>>>>>>>>>>>>>>>>>>>>>>>>>> https://gabycoder88.wordpress.com/2020/11/30/example-post-3/ >>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>> 3. I joined the Outreachy Zulip server for the >>>>>>>>>>>>>>>>>>>>>>>>>>>> chats. >>>>>>>>>>>>>>>>>>>>>>>>>>>> 4. I installed Neo4j on my computer using >>>>>>>>>>>>>>>>>>>>>>>>>>>> docker * . >>>>>>>>>>>>>>>>>>>>>>>>>>>> 5. I drew a class diagram of my project >>>>>>>>>>>>>>>>>>>>>>>>>>>> (attached). >>>>>>>>>>>>>>>>>>>>>>>>>>>> 6. I read the Neo4j documentation about the >>>>>>>>>>>>>>>>>>>>>>>>>>>> Java connector, Neo4j (Neo4j Driver) syntax and >>>>>>>>>>>>>>>>>>>>>>>>>>>> concepts of graph databases. >>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>> * I tried to install native Neo4j, but I >>>>>>>>>>>>>>>>>>>>>>>>>>>> figured that Neo4j requires java 11 and apache >>>>>>>>>>>>>>>>>>>>>>>>>>>> gora requires java 8, hence >>>>>>>>>>>>>>>>>>>>>>>>>>>> I preferred to install using docker to avoid any >>>>>>>>>>>>>>>>>>>>>>>>>>>> issues with the java >>>>>>>>>>>>>>>>>>>>>>>>>>>> version. Is a docker container enough or I must >>>>>>>>>>>>>>>>>>>>>>>>>>>> install natively? >>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>> Regards, >>>>>>>>>>>>>>>>>>>>>>>>>>>> Gaby >>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>