[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #250 was SUCCESSFUL (with 1263 tests)

--- Spring Data GemFire > Nightly-ApacheGeode > #250 was successful. --- Scheduled 1267 tests in total. https://build.spring.io/browse/SGF-NAG-250/ -- This

Review Request 44821: GEODE-1099: NPE thrown from TXManagerImpl.isDistributed()

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44821/ --- Review request for geode and Darrel Schneider. Repository: geode Description

Re: Review Request 44818: GEODE-949: refactor test security implementation PART 1 of 2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44818/ --- (Updated March 15, 2016, 12:56 a.m.) Review request for geode, Jens Deppe and J

Review Request 44818: GEODE-949: refactor test security implementation PART 1 of 2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44818/ --- Review request for geode, Jens Deppe and Jinmei Liao. Bugs: GEODE-949 https

Re: Review Request 44802: GEODE-1049 LocatorDUnitTest.testMultipleLocatorsRestartingAtSameTime

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44802/#review123546 --- Ship it! Change looks good. Just one more comment: A brief desc

Re: Review Request 44813: GEODE-1097 allow Lambda invocations to be named so that they show up in test logs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44813/#review123539 --- Ship it! Ship It! - Dan Smith On March 14, 2016, 10:55 p.m.,

Re: Packaging and dependencies

+1 Funny enough I remember creating a story about that in an older tracker talking about the same issues and leveraging the concept we had at that time for gfxd. The problem Fred describes about deploying big jars we have experienced through docker and lattice (microcf) and can be seen in oth

Re: Packaging and dependencies

Exactly. Thanks everyone for chiming in. On Mon, Mar 14, 2016 at 4:12 PM Udo Kohlmeyer wrote: > +1 > > This should also simplify the client and server code base... > > The step after that should be different libraries for different > functionality in the product. If you need WAN, then you add W

Re: Review Request 44813: GEODE-1097 allow Lambda invocations to be named so that they show up in test logs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44813/#review123531 --- Ship it! Ship It! - Udo Kohlmeyer On March 14, 2016, 10:55 p

Re: Review Request 44812: removed old logging code from offheap

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44812/#review123530 --- geode-core/src/main/java/com/gemstone/gemfire/internal/offheap/Fr

Re: Packaging and dependencies

+1 This should also simplify the client and server code base... The step after that should be different libraries for different functionality in the product. If you need WAN, then you add WAN lib and if you need CQ's you can add that lib. We then end up with very modular application and only

Re: Packaging and dependencies

Yep, some work on modularization is already underway: -rw-r--r-- 1 abaker staff 1735 Mar 7 08:44 geode-common-1.0.0-incubating.M2-SNAPSHOT.jar -rw-r--r-- 1 abaker staff 12810411 Mar 7 08:44 geode-core-1.0.0-incubating.M2-SNAPSHOT.jar -rw-r--r-- 1 abaker staff 89629 Mar 7 08:44

Re: Review Request 44813: GEODE-1097 allow Lambda invocations to be named so that they show up in test logs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44813/#review123528 --- Ship it! Ship It! - Hitesh Khamesra On March 14, 2016, 10:55

Re: Packaging and dependencies

+1 to splitting things up with a view making a smaller client and core jar. BTW, I'm currently working on GEODE-27. One of the things I'm doing is marking a bunch of the dependencies such as jetty as optional in our pom. But that's not the real fix. The fix is to move the code that depends on thos

Re: Review Request 44802: GEODE-1049 LocatorDUnitTest.testMultipleLocatorsRestartingAtSameTime

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44802/#review123526 --- Ship it! Ship It! - Udo Kohlmeyer On March 14, 2016, 7:19 p.

Re: Review Request 44802: GEODE-1049 LocatorDUnitTest.testMultipleLocatorsRestartingAtSameTime

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44802/#review123525 --- Ship it! Ship It! - Bruce Schuchardt On March 14, 2016, 7:19

Review Request 44813: GEODE-1097 allow Lambda invocations to be named so that they show up in test logs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44813/ --- Review request for geode, Hitesh Khamesra, Jianxia Chen, Udo Kohlmeyer, and Dan

Re: Packaging and dependencies

Adding to that, for mobile apps our recommendation was to use REST APIs. -Anil. On Mon, Mar 14, 2016 at 3:51 PM, Anilkumar Gingade wrote: > Hi Fred, > > You are right, with micro-services and mobile applications getting > prominent there is a need to separate our client code...This had came up

Re: Packaging and dependencies

Manuel, thanks for weighting in. Okay, this definitely needs to make its way into our roadmap to be prioritized soon. any other thoughts? On Mon, Mar 14, 2016 at 3:51 PM Anilkumar Gingade wrote: > Hi Fred, > > You are right, with micro-services and mobile applications getting > prominent there

Re: Packaging and dependencies

Hi Fred, You are right, with micro-services and mobile applications getting prominent there is a need to separate our client code...This had came up multiple times in our product road-map (previously)...but never got prioritized as no customer pushed for this... -Anil. On Mon, Mar 14, 2016 at

Review Request 44812: removed old logging code from offheap

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44812/ --- Review request for geode, Kirk Lund and Sai Boorlagadda. Repository: geode De

RE: Packaging and dependencies

Hi Fred, I totally agree to that request. We should avoid to break this into a hell of package dependencies like server, core, client, ... But a more friendly packaging would be great. I would also vote for two packages first and the go from there. Best Regards Manuel Birke Mercedes-Benz Rese

Packaging and dependencies

Folks, Data Microservices use cases are getting more and more popular and libraries like SpringBoot + Geode can be a pretty powerful combo. However, there are serious challenges on how we package and distribute Geode through its maven repository (org.apache.geode:gemfire-core:1.0.0-incuba

[GitHub] incubator-geode pull request: GEODE-52: Remove @author tags from J...

Github user sboorlagadda commented on the pull request: https://github.com/apache/incubator-geode/pull/100#issuecomment-196535333 Closing this pull-request as this work is re-done due to name change for sub-projects from Gemfire to Geode and the new re-work in on feature/GEODE-52 bran

[GitHub] incubator-geode pull request: GEODE-52: Remove @author tags from J...

Github user sboorlagadda closed the pull request at: https://github.com/apache/incubator-geode/pull/100 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

Re: Broken: apache/incubator-geode#116 (feature/GEODE-1050 - 2fd9cc6)

I'm already using Jenkins to run check on my feature branches, so I don't really get any value from Travis. If it uses AWS or other $ resource, then I would suggest not running it automatically on feature branches. Perhaps there could be a way to manually initiate it rather than automatically runni

Review Request 44802: GEODE-1049 LocatorDUnitTest.testMultipleLocatorsRestartingAtSameTime

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44802/ --- Review request for geode, Bruce Schuchardt, Jianxia Chen, and Udo Kohlmeyer. Re

Re: Broken: apache/incubator-geode#116 (feature/GEODE-1050 - 2fd9cc6)

Well, Travis runs build (./gradlew check) on pull requests as well as all branches. Do we want to skip running travis on all feature branches? On Mon, Mar 14, 2016 at 9:00 AM, Anthony Baker wrote: > Ah, thanks for pointing that out Kirk, I missed the branch. > > I think this is setup so we get t

Re: Broken: apache/incubator-geode#116 (feature/GEODE-1050 - 2fd9cc6)

Ah, thanks for pointing that out Kirk, I missed the branch. I think this is setup so we get test coverage on pull requests. Anthony > On Mar 14, 2016, at 8:55 AM, Kirk Lund wrote: > > That branch isn't quite ready to merge to develop, but I did add a category > to that test on Friday. > > Is

Re: Broken: apache/incubator-geode#116 (feature/GEODE-1050 - 2fd9cc6)

That branch isn't quite ready to merge to develop, but I did add a category to that test on Friday. Is there some reason we're running Travis against feature branches that are under active development? I commit/push in-progress work to facilitate testing on both my Mac and Linux. -Kirk On Monda

Re: Broken: apache/incubator-geode#116 (feature/GEODE-1050 - 2fd9cc6)

:geode-core:checkMissedTestsThe test com.gemstone.gemfire.test.dunit.internal.tests.JUnit4BasicDUnitTest.testInvokeLambdaAsync does not include a junit category. <>org.gradle.api.GradleException: The test com.gemstone.gemfire.test.dunit.internal.tests.JUnit4BasicDUnitTest.testInvokeLambdaAsync

Re: Review Request 44736: GEODE-1078 CI failure: ClientMembershipSelectorDUnitTest.testClientMembershipEventsInServer

> On March 13, 2016, 8:17 p.m., Udo Kohlmeyer wrote: > > geode-core/src/test/java/com/gemstone/gemfire/cache30/ClientMembershipDUnitTest.java, > > line 172 > > > > > > Any reason why we replace logger with System.ou

Jenkins build is back to normal : Geode-nightly #408

See

Re: Review Request 44737: GEODE-1084 strip copyright from jgroups configuration before parsing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44737/#review123363 --- Ship it! Ship It! - Udo Kohlmeyer On March 11, 2016, 10:41 p