+1 to merge! I went quickly over the code and looks good to me. I think you guys already commented the most pressing issues. Thanks!
Renato M. 2017-07-27 8:13 GMT-07:00 Furkan KAMACI <[email protected]>: > +1 to merge! > > On Wed, Jul 26, 2017 at 7:13 PM, Nishadi Kirielle < > [email protected] > > wrote: > > > Thanks Kevin. > > > > I will create tickets for the remaining work. > > > > Regards > > Nishadi > > > > On 26 July 2017 at 21:08, Kevin Ratnasekera <[email protected]> > > wrote: > > > >> Hi Lewis, > >> > >> Will do it Lewis, will wait others to comment if any case as you said, > >> otherwise will proceed merging to master. > >> > >> @Nishadi Can you please create tickets for the existing work? Forgot to > >> mention, this is very good work :) > >> > >> Regards > >> Kevin > >> > >> On Wed, Jul 26, 2017 at 8:55 PM, Nishadi Kirielle < > >> [email protected]> wrote: > >> > >>> Thanks a lot Lewis. > >>> > >>> On 26 July 2017 at 20:54, lewis john mcgibbney <[email protected]> > >>> wrote: > >>> > >>>> Kevin or Furkan, if you could, please wait around 24 hours or so for > any > >>>> further code reviews and then go ahead and merge into master branch. > >>>> Good work Nishadi, very good work. > >>>> > >>>> On Wed, Jul 26, 2017 at 8:19 AM, Kevin Ratnasekera < > >>>> [email protected]> > >>>> wrote: > >>>> > >>>> > +1 > >>>> > > >>>> > On Wed, Jul 26, 2017 at 8:48 PM, lewis john mcgibbney < > >>>> [email protected]> > >>>> > wrote: > >>>> > > >>>> >> Thnx for update. The code is looking pretty solid. > >>>> >> Folks, what are you throughts on merging this into master branch? > We > >>>> can > >>>> >> then create separate tickets for the following issues > >>>> >> > >>>> >> - The waiting strategy of the docker client > >>>> >> - Incorporate the map reduce based test cases > >>>> >> - Improve the user documentation > >>>> >> > >>>> >> ??? > >>>> >> > >>>> >> > >>>> >> On Wed, Jul 26, 2017 at 8:04 AM, Nishadi Kirielle < > >>>> >> [email protected]> wrote: > >>>> >> > >>>> >>> Hi all, > >>>> >>> > >>>> >>> As of the second evaluation period, I was able to implement most > of > >>>> the > >>>> >>> possible functionality on gora-aerospike module. > >>>> >>> > >>>> >>> Currently implemented features[1] include the follows; > >>>> >>> > >>>> >>> - Basic PUT, GET, DELETE functionality > >>>> >>> - Querying the whole set of data / querying on a single key > >>>> >>> - Delete functionality based on queries > >>>> >>> - 25 test cases got passed from the base test case > >>>> >>> > >>>> >>> The documentation related to the gora-tutorial based on the > >>>> Aerospike > >>>> >>> module can be found here. [2] In addition, the weekly reports can > >>>> be found > >>>> >>> in [3]. > >>>> >>> > >>>> >>> The limitations occurred when incorporating the Aerospike java > >>>> client > >>>> >>> are as follows; > >>>> >>> > >>>> >>> - Schema creation, deletion related functionality are not > >>>> supported > >>>> >>> by Aerospike > >>>> >>> - Querying based on key ranges are not directly supported by > >>>> >>> Aerospike > >>>> >>> > >>>> >>> The plan for the next period is to improve the following features; > >>>> >>> > >>>> >>> - The waiting strategy of the docker client > >>>> >>> - Incorporate the map reduce based test cases > >>>> >>> - Improve the user documentation > >>>> >>> > >>>> >>> Please let me know if anything needs to be improved in these areas > >>>> and > >>>> >>> it will be great to have feedback on the implementation. > >>>> >>> > >>>> >>> [1]. https://github.com/apache/gora/pull/111 > >>>> >>> [2]. https://docs.google.com/document/d/1hS0PrAXVRJ-NWkJyTnq > >>>> >>> Fd_GJ7IBe4AtrUQJOP-QkwXg/edit > >>>> >>> [3]. https://cwiki.apache.org/confluence/display/GORA/Implem > >>>> >>> ent+Aerospike+Datastore+%28GORA-502%29+Reports > >>>> >>> > >>>> >>> Thank you and regards > >>>> >>> Nishadi > >>>> >>> > >>>> >>> > >>>> >>> -- > >>>> >>> Nishadi Kirielle > >>>> >>> > >>>> >>> Undergraduate > >>>> >>> University of Moratuwa - Sri Lanka > >>>> >>> > >>>> >>> Mobile : +94 70 204 5934 <+94%2070%20204%205934> > >>>> >>> Blog : nishadikirielle.wordpress.com > >>>> >>> > >>>> >> > >>>> >> > >>>> >> > >>>> >> -- > >>>> >> http://home.apache.org/~lewismc/ > >>>> >> @hectorMcSpector > >>>> >> http://www.linkedin.com/in/lmcgibbney > >>>> >> > >>>> > > >>>> > > >>>> > >>>> > >>>> -- > >>>> http://home.apache.org/~lewismc/ > >>>> @hectorMcSpector > >>>> http://www.linkedin.com/in/lmcgibbney > >>>> > >>> > >>> > >>> > >>> -- > >>> Nishadi Kirielle > >>> > >>> Undergraduate > >>> University of Moratuwa - Sri Lanka > >>> > >>> Mobile : +94 70 204 5934 <+94%2070%20204%205934> > >>> Blog : nishadikirielle.wordpress.com > >>> > >> > >> > > > > > > -- > > Nishadi Kirielle > > > > Undergraduate > > University of Moratuwa - Sri Lanka > > > > Mobile : +94 70 204 5934 <+94%2070%20204%205934> > > Blog : nishadikirielle.wordpress.com > > >
