Re: Region Put errors out in Local Multi Server environment.

2016-09-23 Thread Dan Smith
Hi Jun, Your perspective coming coming from the HDFS world makes sense with geode as well. You configure the same partitioned region on all of your geode nodes. Geode will then place partition your data across those nodes. Your client will figure out where it should connect to get to the data

Review Request 52229: GEODE-1934: Removing spring-core dependencies from geode-core

2016-09-23 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52229/ --- Review request for geode, Anthony Baker and Hitesh Khamesra. Repository: geode

Re: Region Put errors out in Local Multi Server environment.

2016-09-23 Thread jun aoki
I maybe making some stupid question but I'm from HDFS world where I don't have to care where data's physical location is. (You just connect to Namenode and you are good.) Bear with me. On Fri, Sep 23, 2016 at 5:26 PM, jun aoki wrote: > Hi Dan, thank you for sharing your

Re: Hibernate module and Geode 1.0 ?

2016-09-23 Thread Anthony Baker
Likewise! Geode provides an L2 cache for Hibernate. That is, an application that is using Hibernate could plug in Geode for caching. Specifically, we implement Hibernate’s cache interfaces like CacheProvider, RegionFactory, etc. There are build-time dependencies on several hibernate jars

Re: Review Request 52176: GEODE-1926: Modification of peedAhead() function check if heapCopy is successful before adding the key to peekedIds

2016-09-23 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52176/#review150271 --- Ship it! Ship It! - Dan Smith On Sept. 23, 2016, 9:58 p.m.,

Re: Hibernate module and Geode 1.0 ?

2016-09-23 Thread Dan Smith
Geode provides an optional L2 cache for hibernate: http://geode.docs.pivotal.io/docs/tools_modules/hibernate_cache/setting_up_the_module.html Looking at the code, it looks like the geode-modules-hibernate is pulling in hibernate-core as a compile dependency, but it's not being distributed as

Re: Region Put errors out in Local Multi Server environment.

2016-09-23 Thread Goutam Tadi
Thanks a lot Dan :-). Yeah, that was intentional. Your solution solves my problem. Thanks, Goutam Tadi. On Fri, Sep 23, 2016 at 4:58 PM Dan Smith wrote: > Hi Goutam, > > It looks like you configured your two servers to have different regions. > Was that intentional? What's

Re: Region Put errors out in Local Multi Server environment.

2016-09-23 Thread Dan Smith
Hi Goutam, It looks like you configured your two servers to have different regions. Was that intentional? What's happening is that the client is connecting to only one of the servers, which has one of your regions but not the other. Generally, when you configure gemfire servers, you should

Re: Build failed in Jenkins: Geode-spark-connector #78

2016-09-23 Thread Anthony Baker
Yep, I’m seeing failures on any client app that doesn’t explicitly include spring as dependency. Exception in thread "main" java.lang.NoClassDefFoundError: org/springframework/util/StringUtils at

Region Put errors out in Local Multi Server environment.

2016-09-23 Thread Goutam Tadi
Hi, I was facing the *"Region not found"* exception when I do the following on local machine (single Node) : And, I don't see the exception when I was trying to perform remote debug which introduced some time lapse. I tried to introduce some `sleep` , but of no use Can you please help and let me

Re: Build failed in Jenkins: Geode-spark-connector #78

2016-09-23 Thread Dan Smith
I created GEODE-1934 for this. It looks like the problem is actually that our dependencies for geode-core are messed up. spring-core is marked optional, but we're using it in critical places like this SSLConfigurationFactory. In my opinion we shouldn't depend on spring-core at all unless we're

Review Request 52228: fix hang introduced by GEODE-1885

2016-09-23 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52228/ --- Review request for geode, anilkumar gingade, Eric Shu, Scott Jewell, and Ken

Re: Review Request 52176: GEODE-1926: Modification of peedAhead() function check if heapCopy is successful before adding the key to peekedIds

2016-09-23 Thread Dan Smith
> On Sept. 22, 2016, 11:02 p.m., Dan Smith wrote: > > geode-core/src/main/java/org/apache/geode/internal/cache/wan/serial/SerialGatewaySenderQueue.java, > > line 796 > > > > > > Minor nit - can't you just write

Re: Review Request 52176: GEODE-1926: Modification of peedAhead() function check if heapCopy is successful before adding the key to peekedIds

2016-09-23 Thread nabarun nag
> On Sept. 22, 2016, 11:02 p.m., Dan Smith wrote: > > geode-core/src/main/java/org/apache/geode/internal/cache/wan/serial/SerialGatewaySenderQueue.java, > > line 796 > > > > > > Minor nit - can't you just write

Re: Review Request 52176: GEODE-1926: Modification of peedAhead() function check if heapCopy is successful before adding the key to peekedIds

2016-09-23 Thread nabarun nag
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52176/#review150244 --- I was able to find paralleWANConflationDUnitTest but no

Re: Review Request 52176: GEODE-1926: Modification of peedAhead() function check if heapCopy is successful before adding the key to peekedIds

2016-09-23 Thread nabarun nag
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52176/ --- (Updated Sept. 23, 2016, 9:58 p.m.) Review request for geode, Barry Oglesby,

Re: Review Request 52176: GEODE-1926: Modification of peedAhead() function check if heapCopy is successful before adding the key to peekedIds

2016-09-23 Thread Dan Smith
> On Sept. 22, 2016, 11:02 p.m., Dan Smith wrote: > > geode-core/src/main/java/org/apache/geode/internal/cache/wan/serial/SerialGatewaySenderQueue.java, > > line 796 > > > > > > Minor nit - can't you just write

Re: Review Request 52176: GEODE-1926: Modification of peedAhead() function check if heapCopy is successful before adding the key to peekedIds

2016-09-23 Thread nabarun nag
> On Sept. 23, 2016, 8:29 p.m., Jason Huynh wrote: > > geode-core/src/main/java/org/apache/geode/internal/cache/wan/serial/SerialGatewaySenderQueue.java, > > line 796 > > > > > > +1 on the less confusing part

Re: Review Request 52176: GEODE-1926: Modification of peedAhead() function check if heapCopy is successful before adding the key to peekedIds

2016-09-23 Thread nabarun nag
> On Sept. 22, 2016, 11:02 p.m., Dan Smith wrote: > > Looks like a good fix for a really complicated issue! > > > > I had a couple of minor nitpicks, below. Also, is there a reason why you > > did not remove the code to > > ((GatewaySenderEventImpl)object).makeHeapCopyIfOffHeap() in > >

Re: support for old clients

2016-09-23 Thread Anthony Baker
What about geode-client-support? Or geode-client-compatibility? I think that expresses intent to support backwards compatibility for prior client versions accordingly to the policies in the wiki [1]. Anthony [1] https://cwiki.apache.org/confluence/display/GEODE/Managing+Backward+Compatibility

Re: Review Request 52176: GEODE-1926: Modification of peedAhead() function check if heapCopy is successful before adding the key to peekedIds

2016-09-23 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52176/#review150201 --- Not sure what an appropriate test would be for this, maybe create

support for old clients

2016-09-23 Thread Bruce Schuchardt
I'm creating a new subproject for supporting old GemFire clients and WAN sites and need a name for it. All of the current ones are prefixed with "geode-". How about geode-gemfire-support?

Re: security properties in the cluster config

2016-09-23 Thread Bruce Schuchardt
SSL settings and the new UDP dhAlgo setting can't be in the cluster config. The cluster config is received over TCP/IP so you would have to use unsecured information to retrieve the settings, and you'd have to do it before the cache is created. Does the security-manager have any role to play

Re: security properties in the cluster config

2016-09-23 Thread Michael Stolz
I am in favor of keeping the SSL thoughts separate from the RBAC thoughts, but I don't see any reason they couldn't share the same repository. That said though, does putting it all into the Cluster Configuration Manager (CCM) make it so that you can only have security if you are using CCM for

security properties in the cluster config

2016-09-23 Thread Jinmei Liao
Hi, All, I am working on this ticket: https://issues.apache.org/jira/browse/GEODE-1659. Basically, currently, any member(locator or server) needs to specify its own security-manager in order to protect its data which could leads to misconfiguration and data leak. So we would like to put it into

Re: jvsd

2016-09-23 Thread Michael Stolz
+1 If its not part of a release its documentation shouldn't be either. -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: 631-835-4771 On Thu, Sep 22, 2016 at 7:58 PM, Joey McAllister wrote: > If it isn't on develop or planned for release, then I vote for

Re: jvsd

2016-09-23 Thread Joey McAllister
+1 Thanks, Dave! On Fri, Sep 23, 2016 at 9:37 AM Dave Barnes wrote: > Good observations, John. > A clarification: JVSD is Geode-only. The corresponding GemFire tool, VSD, > enjoys a more 'mainstream' status, at least for now, so it still appears in > the GF user guide. > >

Re: jvsd

2016-09-23 Thread Dave Barnes
Good observations, John. A clarification: JVSD is Geode-only. The corresponding GemFire tool, VSD, enjoys a more 'mainstream' status, at least for now, so it still appears in the GF user guide. On Fri, Sep 23, 2016 at 9:34 AM, John Blum wrote: > +1 > > Even a bit of

Re: jvsd

2016-09-23 Thread John Blum
+1 Even a bit of documentation (which seems scattered about... in the specification, on VMW sites/properties, etc) would go a long way in helping users realize the benefit of the tool and provide feedback, maybe even contribute some PRs. Having metrics on GemFire in realtime is hugely invaluable

Re: jvsd

2016-09-23 Thread Dave Barnes
Here's a proposal based on what I've seen in this thread: 1. We remove JVSD documentation from the user manual. 2. We save what's been written so far (mostly just build instructions) as a README on the feature branch. On Thu, Sep 22, 2016 at 7:40 PM, Anthony Baker wrote: >

Build failed in Jenkins: Geode-spark-connector #78

2016-09-23 Thread Apache Jenkins Server
See Changes: [hkhamesra] GEODE-37 In spark connector we call TcpClient static method to get the [klund] GEODE-1906: fix misspelling of Successfully [upthewaterspout] GEODE-1915: Prevent deadlock registering instantiators with