Re: Merging GEODE-9 Spark Connector project into develop

2015-07-24 Thread Qihong Chen
+1 to merge On Fri, Jul 24, 2015 at 9:54 AM, Dan Smith wrote: > +1 to merge. > > -Dan > > On Thu, Jul 23, 2015 at 11:44 PM, Jianxia Chen wrote: > > > +1 for merge > > > > On Thu, Jul 23, 2015 at 4:27 PM, Anilkumar Gingade > > wrote: > > > > > +1 for merge. > > > > > > On Thu, Jul 23, 2015 at 1

Re: Review Request 36731: GEODE-9 : Merging in spark connector sub project

2015-07-23 Thread Qihong Chen
forgot to ask, do we need .gitignore file? there's only one line of "/bin/" in it. Thanks! ​

Re: Review Request 36731: GEODE-9 : Merging in spark connector sub project

2015-07-23 Thread Qihong Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36731/#review92775 --- Ship it! Ship It! - Qihong Chen On July 23, 2015, 4:29 p.m

Re: Review Request 36731: GEODE-9 : Merging in spark connector sub project

2015-07-23 Thread Qihong Chen
I'm not talking about changes to GEODE-9 feature branch. Just want to know if this merge is just "copy-ing" all files from GEODE-9 feature branch to develop branch? Thanks! ​

Re: Review Request 36731: GEODE-9 : Merging in spark connector sub project

2015-07-23 Thread Qihong Chen
Hi Jason, Is there any "real" changes from GEODE-9 feature branch except adding all files to develop branch? Thanks, Qihong ​

Re: Review Request 36408: Resolve the Geode Spark Connector Build Issue

2015-07-14 Thread Qihong Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36408/#review91625 --- Ship it! Ship It! - Qihong Chen On July 10, 2015, 11:32 p.m

Re: Where to place "Spark + GemFire" connector.

2015-07-06 Thread Qihong Chen
The problem is caused by multiple major dependencies and different release cycles. Spark Geode Connector depends on two products: Spark and Geode (not counting other dependencies), and Spark moves much faster than Geode, and some features/code are not backward compatible. Our initial connector imp