Re: Orphaned Server processes

2017-04-05 Thread Udo Kohlmeyer
the server and register interest, execute queries, function -Anil. On Wed, Apr 5, 2017 at 11:03 AM, Udo Kohlmeyer wrote: @Anil, I agree... quick start to evaluate the product... "start server --name=server1" is the simplest way to start a server... BUT there are no regions o

Re: Orphaned Server processes

2017-04-05 Thread Udo Kohlmeyer
ster the server. --Udo On 4/5/17 15:25, Anilkumar Gingade wrote: Udo, I was commenting on the statement: Current Geode allows a user to start a server without being linked to a Locator. Not specifically starting server with gfsh... -Anil. On Wed, Apr 5, 2017 at 3:18 PM, Udo Kohlmeyer

Re: Orphaned Server processes

2017-04-05 Thread Udo Kohlmeyer
erver1" as you state, it may still find a cache.xml to configure itself. I'm not saying this is better than using GFSH to configure the server, but it exists today and your proposal should explain how and why the search pattern for cache.xml is going away or something like that if you do i

Re: Review Request 58155: GEODE-2653: fix a flaky test.

2017-04-06 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58155/#review171212 --- Ship it! Ship It! - Udo Kohlmeyer On April 5, 2017, 11:44

Re: Review Request 58187: GEODE-2732 after auto-reconnect a server is restarted on the default port of 40404

2017-04-06 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58187/#review171213 --- Ship it! Ship It! - Udo Kohlmeyer On April 4, 2017, 8:46

Re: Review Request 56586: GEODE-1716: refactor JMXMBeanDunitTest

2017-04-06 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56586/#review171214 --- Ship it! Ship It! - Udo Kohlmeyer On Feb. 13, 2017, 4:07

Re: Review Request 58484: GEODE-2632: refactoring part 2

2017-04-18 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58484/#review172210 --- Ship it! Ship It! - Udo Kohlmeyer On April 17, 2017, 10:27

Re: Review Request 58541: GEODE-576 & GEODE-516 Flaky test: GemFireDeadlockDetectorDUnitTest.testDistributedDeadlockWithFunction

2017-04-20 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58541/#review172557 --- Ship it! Ship It! - Udo Kohlmeyer On April 19, 2017, 8:27

Re: Simple Java Client

2017-04-24 Thread Udo Kohlmeyer
+1 for calling it a driver :) On 4/24/17 16:07, Michael William Dodge wrote: Perhaps I'm picking nits, but I think a library that provides an API for interacting with Geode isn't a client. (I like to call it a driver.) The client is the application that someone write to use that API to intera

Re: errors from logging code in unit tests

2021-02-24 Thread Udo Kohlmeyer
What test(s) are you running? Version? --Udo On 2/25/21, 8:39 AM, "Ernie Burghardt" wrote: Its not just you Bruce, I get this... 2021-02-24 13:38:10,891 main ERROR Error processing element GeodeConsole ([Appenders: null]): CLASS_NOT_FOUND 2021-02-24 13:38:10,898 main ERROR Error p

Re: [DISCUSS] removal of experimental Protobuf client/server interface

2021-03-23 Thread Udo Kohlmeyer
-1 Given that I was on the team that started this initiative, I will naturally have an inclination to say 'No'. I don't know if I would go as far as removing this project/initiative out of Geode. I understand that the way that was used to hook into Geode was less the perfect, and I fully supp

Re: [DISCUSS] removal of experimental Protobuf client/server interface

2021-03-23 Thread Udo Kohlmeyer
eserve their own dedicated discussion. I understand some of them are harder to remove from a technical perspective or neither experimental nor deprecated and would thus require a Geode 2.0. ________ From: Udo Kohlmeyer Sent: Tuesday, March 23, 2021 14:54 To: dev@geo

Re: [DISCUSS] removal of experimental Protobuf client/server interface

2021-03-31 Thread Udo Kohlmeyer
ware.com%7C7a18231640654f0546fc08d8f4651dc0%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637528062421752074%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=BO8%2BENOPc05J%2BlA4Wu4O9kIyVetnK4xihMfEMaQ%2FbBs%3D&reserved=0>

Re: [VOTE] Requiring final keyword on every parameter and local variable

2021-04-14 Thread Udo Kohlmeyer
"+1" to ENCOURAGING developers to make "final" a requirement for method arguments. "-1" to making it a hard rule. If we want to enforce this rule on the basis of readability or performance, I fear that we might be beating the wrong horse here for the wrong reasons!! Short reasoning: If we rea

Re: [VOTE] Requiring final keyword on every parameter and local variable

2021-04-15 Thread Udo Kohlmeyer
provements? --Udo On 4/15/21, 1:19 PM, "Jacob Barrett" wrote: > On Apr 14, 2021, at 7:46 PM, Udo Kohlmeyer wrote: > @Jake the idea of smaller methods is great and we should ALWAYS strive for that. But that argument is completely irrelevant in this discussion. As mak

Re: Rescued Geode Protobuf

2021-05-05 Thread Udo Kohlmeyer
Thank you Michael, I’ll definitely be looking into this in the future… --Udo From: Michael Oleske Date: Thursday, May 6, 2021 at 9:22 AM To: dev@geode.apache.org Subject: Rescued Geode Protobuf Hi Geode Friends! Since I didn't see any movement on rescuing Geode Protobuf into another reposito

Re: Cleaning up the codebase - use curly braces everywhere

2021-05-27 Thread Udo Kohlmeyer
+1, I trust the work that you’ve done @Donal. I think splitting this out by package or any other metric we could come up with, it an exercise in futility. So let’s get this merged in, set up a spotless rule to make sure we don’t introduce more and move on… --Udo From: Donal Evans Date: Frida

Access to the CI pipelines using fly

2021-07-01 Thread Udo Kohlmeyer
Hi there Apache Geode Infra, Would it be possible to please grant me access to fly access of a CI pipeline. I’m trying to root-cause a problem which I see happen only in the CI-pipeline. --Udo

Re: Access to the CI pipelines using fly

2021-07-01 Thread Udo Kohlmeyer
Thank you From: Robert Houghton Date: Friday, July 2, 2021 at 7:03 AM To: dev@geode.apache.org Subject: Re: Access to the CI pipelines using fly Sure thing, Udo. I have updated the concourse instance with your github name. -Robert From: Udo Kohlmeyer Date: Thursday, July 1, 2021 at 1:44 PM

[DISCUSS] - Creation of new Apache Geode Github repo for Gradle Plugin

2021-07-20 Thread Udo Kohlmeyer
Hi there Apache Dev. As part of Geode-8705, there has been some work done on creating a gradle plugin that will generate module.xml files that are consumed by JBoss Modules, based on the gradle build files. This plugin is intended to be not only used by the Geode project but also for customers

Re: [DISCUSS] Upgrading to Lucene 7.1.0

2021-09-27 Thread Udo Kohlmeyer
Might I propose something here. There is currently a significant amount of work going into completing Geode-8705, which is the Classloader isolation. We are currently targeting to getting this release in Geode 1.16. My proposal is, that we use the capability that Patrick demo’d at the Community

Re: Region is not created on one of the servers

2021-10-11 Thread Udo Kohlmeyer
Hi Mario, I think your assessment of the problem is correct. Thinking about it, there is no simple (correct) way to easily solve this. Given that there are too many variables in play, users making configurational changes, whilst servers are coming up. Now, that said, I think we should address

Geode-9621: Spotless overstepping its boundary of formatter

2021-10-11 Thread Udo Kohlmeyer
Hi there Geode Devs, Recently GEODE-9621 was introduced onto the develop branch. This new feature introduces a new bug where `spotlessApply` will remove all version dependencies out of gradle. I assume this is to combat the risin

Re: Geode-9621: Spotless overstepping its boundary of formatter

2021-10-11 Thread Udo Kohlmeyer
Hi there Robert, Yes, you did work with me, but the solution to add my custom versions of jars that I want to test with, into the `DependencyConstraints` file just feels like the wrong approach. The prior art you refer to, is a specific version of tomcat we want each module to use. It is no di

Re: Geode-9621: Spotless overstepping its boundary of formatter

2021-10-12 Thread Udo Kohlmeyer
against that. -Robert From: Udo Kohlmeyer Date: Monday, October 11, 2021 at 5:32 PM To: dev@geode.apache.org Subject: Re: Geode-9621: Spotless overstepping its boundary of formatter Hi there Robert, Yes, you did work with me, but the solution to add my custom versions of jars that I want to test

Re: Geode-9621: Spotless overstepping its boundary of formatter

2021-10-14 Thread Udo Kohlmeyer
specify multiple versions at once, and that tests would be out-of-sync with each other, and with the production code. So I would strongly vote against that. -Robert From: Udo Kohlmeyer Date: Monday, October 11, 2021 at 5:32 PM To: dev@geode.apache.org Subject: Re: Geode-9621: Spotless

Re: [DISCUSS] batch patch release voting

2021-12-02 Thread Udo Kohlmeyer
I wonder if voting on patch versions makes sense. As we should never be breaking any existing features and essentially there should be sufficient testing on the fixes to confirm that they resolve the issues. There should also be no changes to APIs, as those changes should be included in a new mi

Re: [DISCUSS] batch patch release voting

2021-12-02 Thread Udo Kohlmeyer
Unknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=ISRlc%2BXuR%2F9kNGbWoejBjx5stGD0wfAuoxDmaMj1tx4%3D&reserved=0> On December 2, 2021 at 3:25:03 PM PST, Udo Kohlmeyer wrote: I wonder if voting on patch versions makes sense. As we should never be breaking any existing features and essential

Re: Proposal: Cutting 1.15 Release branch Tuesday, 25 January

2022-01-20 Thread Udo Kohlmeyer
+1 on cutting a release on 25 Jan 2022… I implore all community members, that if you have a feature that has not met the 25 Jan 2022 deadline, to not try and add it post the deadline. This will help in the final stabilization phase of the release. --Udo From: Raymond Ingles Date: Friday, Janu

Re: [Suspected Spam] [VOTE] Apache Geode 1.14.3.RC1

2022-01-24 Thread Udo Kohlmeyer
+1 – Ran basic acceptance tests. From: Dave Barnes Date: Tuesday, January 25, 2022 at 6:12 AM To: dev@geode.apache.org Subject: Re: [Suspected Spam] [VOTE] Apache Geode 1.14.3.RC1 +1 docs binary distribution contains API docs with correct version header source distribution provides working scrip

Re: Commit message review opt-in

2022-03-04 Thread Udo Kohlmeyer
I’ll opt-in please. --Udo From: Owen Nichols Date: Saturday, February 26, 2022 at 10:44 AM To: geode Subject: Commit message review opt-in If you received an automated-looking PR review comment from onichols-pivotal in the past few weeks without consent, I apologize and have withdrawn it. If

Re: [Proposal] Make "Affects Version" field mandatory for new tickets with "Bug" issue type in JIRA

2022-05-26 Thread Udo Kohlmeyer
I think it is reasonable to “reject” bug reports that don’t include simple things like “version” and “description”. I agree, a bug report with the initially effected version filled in should be marked as incomplete and the same should be said about the description field. If there is not enough i

Re: [Proposal] Make "Affects Version" field mandatory for new tickets with "Bug" issue type in JIRA

2022-05-27 Thread Udo Kohlmeyer
oint about the Description field being necessary. Do you think that this field should also be mandatory for all issue types? Personally, I think there's a fairly strong argument to be made that a ticket should always have a description, so I'd be in favour of that change too. __

Re: [PROPOSAL] Relocate Geode Docs from code repo to seperate repo

2022-06-14 Thread Udo Kohlmeyer
Hi there Dave, I can understand the frustration that you face. I think the freezing of the code is different to that of the docs. I think each project member would agree if I stated that changes to the docs on ANY branch should be allowed regardless of where in the process of the release the pr

Re: Review Request 53685: GEODE-1955: properly disconnect gfsh session so that it won't leave heartbeat thread around to pollute other tests

2016-11-21 Thread Udo Kohlmeyer
ConnectToLocatorSSLDUnitTest.java (line 128) <https://reviews.apache.org/r/53685/#comment226696> I don't think you should ignore this test. Maybe mark them as flaky, but don't remove them. - Udo Kohlmeyer On Nov. 14, 2016, 8:56 p.m., Jinmei Liao wrote: > > --

Re: Review Request 53864: GEODE-2121 add DLockTest, MembershipTest, ClientServerTest, ClientSubscriptionTest and RestAPITest categories

2016-11-21 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53864/#review156515 --- Ship it! Ship It! - Udo Kohlmeyer On Nov. 17, 2016, 11:07

Re: Review Request 53685: GEODE-1955: properly disconnect gfsh session so that it won't leave heartbeat thread around to pollute other tests

2016-11-21 Thread Udo Kohlmeyer
> On Nov. 21, 2016, 10:37 p.m., Udo Kohlmeyer wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/cli/shell/JmxOperationInvoker.java, > > line 262 > > <https://reviews.apache.org/r/53685/diff/2/?file=1563434#file1563434line262> > > > >

Re: Strange Performance Issue with Large number Region Creation

2016-11-22 Thread Udo Kohlmeyer
@Avinesh, I's be interested in understanding why you need to create 1500 regions. Maybe if you could explain the usecase where 1500 regions are required we could potentially help with another solution. This reminds me of the Oracle Table limitation of 999 columns Where the Oracle support

Re: Strange Performance Issue with Large number Region Creation

2016-11-22 Thread Udo Kohlmeyer
+1 Maybe we can create a JIRA for this. On 11/22/16 11:56, Dan Smith wrote: Seems like fixing that StatisicsMonitor to use a ConcurrentHashSet is a good fix, there's no reason why it should be a list that I can see. As other folks mentioned, there is significant overhead involved in creating

Re: The next release (v1.1.0)

2016-11-23 Thread Udo Kohlmeyer
+1... We all want this project to be the best it can be. On 11/23/16 09:07, Dan Smith wrote: Dor - I don't think any of the active committers are planning on leaving. We're working on this project because we want it to thrive, and graduation is one more thing that will help it grow!

Re: JavaLaunchHelper warning when starting server

2016-11-28 Thread Udo Kohlmeyer
I think this is a problem with Java... I recall that this is a known issue in Java and nothing to worry about. On 11/28/16 14:02, Kirk Lund wrote: I've started seeing the following intermixed with GFSH output when starting servers on a Mac: objc[16455]: Class JavaLaunchHelper is implemented i

Re: Global scope regions

2016-11-28 Thread Udo Kohlmeyer
I think it was never added. I think we keep on favoring shortcuts above correct configuration properties. --Udo On 11/28/16 15:15, Kirk Lund wrote: Looks like global regions cannot be created using GFSH. We should either add support for global scope to GFSH or remove global scope (and the Reg

Re: Global scope regions

2016-11-28 Thread Udo Kohlmeyer
I think we need to add --data-policy and --scope to GFSH. --Udo On 11/28/16 15:39, Kirk Lund wrote: Yep, the --type argument in gfsh matches the enum values of org.apache.geode.cache.RegionShortcut which doesn't include any shortcuts for global. On Mon, Nov 28, 2016 at 3:17 PM

Re: Review Request 54003: GEODE-2137 client membership IDs contain 17 bytes of useless information that should be removed

2016-11-29 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54003/#review157338 --- Ship it! Ship It! - Udo Kohlmeyer On Nov. 29, 2016, 9:12

Re: Review Request 53685: GEODE-1955: properly disconnect gfsh session so that it won't leave heartbeat thread around to pollute other tests

2016-11-30 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53685/#review157416 --- Ship it! Ship It! - Udo Kohlmeyer On Nov. 28, 2016, 11:27

Review Request 54209: GEODE-1931: CI Failure: LocatorUDPSecurityDUnitTest.testStartTwoLocators

2016-11-30 Thread Udo Kohlmeyer
geode-core/src/test/java/org/apache/geode/distributed/LocatorUDPSecurityDUnitTest.java 3246f85 Diff: https://reviews.apache.org/r/54209/diff/ Testing --- Precheckin - completed Thanks, Udo Kohlmeyer

Re: Review Request 54235: GEODE-2159 add serialVersionUIDs to exception classes not having them

2016-12-01 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54235/#review157623 --- Ship it! Ship It! - Udo Kohlmeyer On Nov. 30, 2016, 11:34

Re: Review Request 54542: [GEODE-2191] Add a new randomized string serialization test.

2016-12-08 Thread Udo Kohlmeyer
<https://reviews.apache.org/r/54542/#comment229353> Please provide a better property name. rng does not mean anything to me. - Udo Kohlmeyer On Dec. 8, 2016, 8:26 p.m., Galen O'Sullivan wrote: > > --- > This is an automatically

Re: Review Request 54542: [GEODE-2191] Add a new randomized string serialization test.

2016-12-08 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54542/#review158579 --- Ship it! Ship It! - Udo Kohlmeyer On Dec. 8, 2016, 8:26 p.m

Re: Review Request 54546: GEODE-2193 a member is kicked out immediately after joining

2016-12-08 Thread Udo Kohlmeyer
.java (line 834) <https://reviews.apache.org/r/54546/#comment229372> spelling "done" - Udo Kohlmeyer On Dec. 8, 2016, 10:53 p.m., Bruce Schuchardt wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 54761: GEODE-2215 NPE in ViewCreator thread setting public keys into a NetVie

2016-12-15 Thread Udo Kohlmeyer
/membership/NetView.java (lines 608 - 610) <https://reviews.apache.org/r/54761/#comment230295> Do we maybe not want to clear the previous entries before putAll? Do we want to keep all previous entries? - Udo Kohlmeyer On Dec. 14, 2016, 10:41 p.m., Bruce Schuchardt

Re: Review Request 54761: GEODE-2215 NPE in ViewCreator thread setting public keys into a NetVie

2016-12-15 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54761/#review159320 --- Ship it! Other than the issue raised, ship it - Udo Kohlmeyer

Re: Review Request 54752: GEODE-2213 Deadlock in GMSJoinLeaveJUnitTest

2016-12-15 Thread Udo Kohlmeyer
itTest.java (line 317) <https://reviews.apache.org/r/54752/#comment230317> I'm questioning the sanity of this test. Realistically this could be a valid address, given they have provided a fully qualified hostname, which should realistically be valid. - Udo Kohlmeyer On D

Re: Review Request 54752: GEODE-2213 Deadlock in GMSJoinLeaveJUnitTest

2016-12-15 Thread Udo Kohlmeyer
to be a sane change. - Udo Kohlmeyer On Dec. 14, 2016, 4:41 p.m., Bruce Schuchardt wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 54870: GEODE-2228 FutureResult.get() does not check for cancellation prior to waiting for a result

2016-12-19 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54870/#review159639 --- Ship it! Ship It! - Udo Kohlmeyer On Dec. 19, 2016, 7:14

Re: Review Request 54879: GEODE-1893 remove ability to create a locator without all location services

2016-12-20 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54879/#review159738 --- Ship it! Ship It! - Udo Kohlmeyer On Dec. 19, 2016, 11:26

New proposal for type definitons

2016-12-20 Thread Udo Kohlmeyer
Hey there, I've just completed a new proposal on the wiki for a new mechanism that could be used to define a type definition for an object. https://cwiki.apache.org/confluence/display/GEODE/Custom+External+Type+Definition+Proposal+for+JSON Primarily the new type definition proposal will hopef

Review Request 54945: GEODE-419: Added test to test fail over to javax properties if ssl enabled but relevant properties are not set

2016-12-21 Thread Udo Kohlmeyer
://reviews.apache.org/r/54945/diff/ Testing --- Added test to test scenario. No code was changed for this. Thanks, Udo Kohlmeyer

Re: Review Request 54209: GEODE-1931: CI Failure: LocatorUDPSecurityDUnitTest.testStartTwoLocators

2016-12-21 Thread Udo Kohlmeyer
apache.org/r/54209/#review157432 --- On Nov. 30, 2016, 5:20 p.m., Udo Kohlmeyer wrote: > > --- > This is an automatically generated e-mail. To reply, vi

Re: New proposal for type definitons

2016-12-22 Thread Udo Kohlmeyer
he extra storage cost. -Dan On Tue, Dec 20, 2016 at 4:53 PM, Udo Kohlmeyer wrote: Hey there, I've just completed a new proposal on the wiki for a new mechanism that could be used to define a type definition for an object. https://cwiki.apache.org/confluence/display/GEODE/Custom+ Extern

Re: New proposal for type definitons

2016-12-22 Thread Udo Kohlmeyer
JSON document so that field order would not generated different pdx types? Also when choosing a pdx field type if we always picked a "wider" type then it would reduce the number of types generated because of different field types. On Thu, Dec 22, 2016 at 10:02 AM, Udo Kohlmeyer wrote:

Re: New proposal for type definitons

2016-12-22 Thread Udo Kohlmeyer
field names from the JSON document so that field order would not generated different pdx types? Also when choosing a pdx field type if we always picked a "wider" type then it would reduce the number of types generated because of different field types. On Thu, Dec 22, 2016 at 10:02 AM,

Re: New proposal for type definitons

2016-12-22 Thread Udo Kohlmeyer
ecause of different field types. On Thu, Dec 22, 2016 at 10:02 AM, Udo Kohlmeyer wrote: Hi there Dan, You are correct, the thought is there to add a flag to the registry to indicate that a definition is custom and thus should not conflict with the existing ids. Even if they types were to be st

Re: New proposal for type definitons

2016-12-22 Thread Udo Kohlmeyer
names from the JSON document so that field order would not generated different pdx types? Also when choosing a pdx field type if we always picked a "wider" type then it would reduce the number of types generated because of different field types. On Thu, Dec 22, 2016 at 10:02 AM, Udo Kohlm

Re: Review Request 55017: cleaning up code in a few places

2016-12-27 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55017/#review160189 --- Ship it! Ship It! - Udo Kohlmeyer On Dec. 23, 2016, 6:10

Re: IntelliJ builds broken looking for OQLLexerTokenTypes.java

2016-12-27 Thread Udo Kohlmeyer
I had the same problem. gradle clean build did the trick for me On 12/27/16 13:45, Bruce Schuchardt wrote: Actually neither refreshing from gradle nor creating a new Intellij project worked. I had to go to the "other" tasks under geode-core in the Gradle window and set "generateGrammarSource"

Re: New proposal for type definitons

2016-12-28 Thread Udo Kohlmeyer
ated because of different field types. On Thu, Dec 22, 2016 at 10:02 AM, Udo Kohlmeyer wrote: Hi there Dan, You are correct, the thought is there to add a flag to the registry to indicate that a definition is custom and thus should not conflict with the existing ids. Even if they types were to be

Re: Review Request 55091: GEODE-2257 Client configured to use locator with addPoolServer fails to connect

2016-12-29 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55091/#review160287 --- Ship it! Ship It! - Udo Kohlmeyer On Dec. 29, 2016, 11:11

Re: Off-Heap Annotations

2017-01-03 Thread Udo Kohlmeyer
Hi there Gal, That is not possible. A region is either on-heap or off-heap. --Udo On 1/2/17 13:50, Gal Palmery wrote: Ok, So if I want to have just a certain part of the region off heap and the rest of it on heap - how do I do that? -Original Message- From: Dan Smith [mailto:dsm..

Re: Off-Heap Annotations

2017-01-03 Thread Udo Kohlmeyer
Dor -Original Message- From: Udo Kohlmeyer [mailto:u...@apache.org] Sent: יום ג 03 ינואר 2017 17:32 To: dev@geode.apache.org Subject: Re: Off-Heap Annotations Hi there Gal, That is not possible. A region is either on-heap or off-heap. --Udo On 1/2/17 13:50, Gal Palmery wrote: Ok, So

Re: Cluster Config

2017-01-03 Thread Udo Kohlmeyer
+1 I think that the configurations of all locators should be identical, or at least in terms of a few "critical" properties. One would also need to be able to amend some property changes at runtime, to allow for the changing of configuration without taking all the locators offline. "remote-lo

Re: Review Request 55106: GEODE-1580 ci failure: GemFireDeadlockDetectorDUnitTest.testDistributedDeadlockWithDLock

2017-01-03 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55106/#review160435 --- Ship it! Ship It! - Udo Kohlmeyer On Jan. 3, 2017, 6:03 p.m

Re: Review Request 55074: GEODE-2253 Locator may fail to respond to a valid request

2017-01-03 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55074/#review160436 --- Ship it! Ship It! - Udo Kohlmeyer On Dec. 28, 2016, 5:15

Re: Review Request 55155: GEODE-2155 Auto-reconnect fails with NPE

2017-01-04 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55155/#review160518 --- Ship it! Ship It! - Udo Kohlmeyer On Jan. 3, 2017, 10:55

Re: New proposal for type definitons

2017-01-04 Thread Udo Kohlmeyer
tore the field names in the type registry and the field values in the serialized data. If we give people the option to store and query self describing values, then users with inconsistent json could just use that option and pay the extra storage cost. -Dan On Tue, Dec 20, 2016 at 4:53 PM, Udo

Review Request 55184: GEODE-1294: Overriding cluster-ssl properties does not work for http-service-ssl

2017-01-04 Thread Udo Kohlmeyer
geode/internal/net/SSLConfigurationFactoryJUnitTest.java 4f8d5b8a3 Diff: https://reviews.apache.org/r/55184/diff/ Testing --- Thanks, Udo Kohlmeyer

User permissions for Galen O'Sullivan

2017-01-05 Thread Udo Kohlmeyer
Hi there, Can someone with the relevant powers and permissions add Galen O'Sullivan as a contributor in Apache Jira? --Udo

Re: IntelliJ builds broken looking for OQLLexerTokenTypes.java

2017-01-05 Thread Udo Kohlmeyer
e does NOT fix this for me. Question: why should a "./gradlew clean build" from command-line be required to get IntelliJ to work? -Kirk On Tue, Dec 27, 2016 at 1:47 PM, Udo Kohlmeyer wrote: I had the same problem. gradle clean build did the trick for me On 12/27/16 13:45, Br

Re: IntelliJ builds broken looking for OQLLexerTokenTypes.java

2017-01-05 Thread Udo Kohlmeyer
-Kirk On Tue, Dec 27, 2016 at 1:47 PM, Udo Kohlmeyer wrote: I had the same problem. gradle clean build did the trick for me On 12/27/16 13:45, Bruce Schuchardt wrote: Actually neither refreshing from gradle nor creating a new Intellij project worked. I had to go to the "other&quo

Re: Server accepts clients before its cache is initialized

2017-01-05 Thread Udo Kohlmeyer
Take this scenario as an example. 1) start server 2) Cache initializes and starts cacheserver 3) run custom code to build up meta structures for custom implementation In this scenario, the cache has initialized and the cacheserver has been started. So clients can start interacting with the serv

Re: Server accepts clients before its cache is initialized

2017-01-05 Thread Udo Kohlmeyer
ith wrote: Udo - could your use case be satisfied just by doing somthing like this? 1) gfsh start server --disable-default-server 2) Run your custom code 3) As the last part of your custom code, call Cache.addCacheServer -Dan On Thu, Jan 5, 2017 at 1:17 PM, Udo Kohlmeyer wrote: Take this scenario as

Re: IntelliJ builds broken looking for OQLLexerTokenTypes.java

2017-01-05 Thread Udo Kohlmeyer
ow to get intellij to automatically run that task that sounds like the way to go. -Dan On Thu, Jan 5, 2017 at 12:47 PM, Udo Kohlmeyer wrote: In the newer IntelliJ you can actually have IntelliJ invoke the gradle commands for build/run/build instead of its own internal implementation. --Udo

Re: Server accepts clients before its cache is initialized

2017-01-05 Thread Udo Kohlmeyer
erver with the start server command. On Thu, Jan 5, 2017 at 1:17 PM, Udo Kohlmeyer wrote: Take this scenario as an example. 1) start server 2) Cache initializes and starts cacheserver 3) run custom code to build up meta structures for custom implementation In this scenario, the cache has

Re: Server accepts clients before its cache is initialized

2017-01-05 Thread Udo Kohlmeyer
the embedded services in one or more GemFire/Geode nodes in the cluster. On Thu, Jan 5, 2017 at 1:57 PM, Udo Kohlmeyer wrote: +2 On 1/5/17 13:53, Kirk Lund wrote: Ok #3 makes sense then. It's not that the Cache initialization is broken, it's that some people want to hook in s

Re: Review Request 55236: GEODE-2271 Reduce pdx type id generation for string fields in JSON

2017-01-06 Thread Udo Kohlmeyer
ine 263) <https://reviews.apache.org/r/55236/#comment231910> Maybe we need a test that confirms that we will only generate 2 type definitions. One where the field exists and one where the field does not exist - Udo Kohlmeyer On Jan. 5, 2017, 11:14 p.m.,

Re: copy files to servers

2017-01-06 Thread Udo Kohlmeyer
I think I can see the benefit of this feature. If you have Geode running in the cloud, it is easier to have a single management tool that can copy resource files to all the servers within the cluster. Although I would not see this a feature I'd promote, as it could really be abused, I believ

Re: copy files to servers

2017-01-06 Thread Udo Kohlmeyer
In some ways that is a great idea but sometimes too explicit... Do we expect them to have fine grained jars? Also how do we handle dependencies as a single util class might be used by both a cache-listener and a partition listener... is the expectation that we update the dependent util c

Re: Visualize Geode Metrics/Stats with Grafana Dashboards

2017-01-11 Thread Udo Kohlmeyer
turnkey sounds good... but with a set of scripts, config files and steps to get to final result, docker would just be an option. --Udo On 1/11/17 08:05, Jens Deppe wrote: This is fantastic work! I did something with the same components for a hackday last year and it looked very promising - it

Re: Review Request 55389: GEODE-2282 Provide ability to sort field while creating pdxType for jSON document

2017-01-12 Thread Udo Kohlmeyer
- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55389/ > ----------- > > (Updated Jan. 10, 2017, 7:36 p.m.) > > > Review request for geode, Bruc

Re: Review Request 55389: GEODE-2282 Provide ability to sort field while creating pdxType for jSON document

2017-01-12 Thread Udo Kohlmeyer
/org/apache/geode/pdx/internal/json/PdxInstanceSortedHelper.java (line 321) <https://reviews.apache.org/r/55389/#comment232700> Why would this be a static method? - Udo Kohlmeyer On Jan. 10, 2017, 7:36 p.m., Hitesh Khamesra wrote: > >

Re: Review Request 55389: GEODE-2282 Provide ability to sort field while creating pdxType for jSON document

2017-01-12 Thread Udo Kohlmeyer
fore we do variable length. Might be a "nice" improvement. - Udo Kohlmeyer On Jan. 10, 2017, 7:36 p.m., Hitesh Khamesra wrote: > > --- > This is an automatically generated e-mail. To reply

Re: Review Request 55389: GEODE-2282 Provide ability to sort field while creating pdxType for jSON document

2017-01-12 Thread Udo Kohlmeyer
g/r/55389/#comment232709> Maybe we can have a "getJSONParser(Object input)" method that would return a jsonParser. Then we can merge the two fromJson methods or at least not duplicate the internals. - Udo Kohlmeyer On Jan. 10, 2017, 7:36 p.m.,

Re: Debugging JUnit / DUnit tests

2017-01-13 Thread Udo Kohlmeyer
Have you tried starting the remote with java -agentlib:jdwp=transport=dt_socket,address=127.0.0.1:9001,server=y,suspend=y –jar application.jar and then you should be able to have your Idea connect to it using the "Remote" configuration in your run/debug configurations. Never tried this...

Re: Debugging JUnit / DUnit tests

2017-01-13 Thread Udo Kohlmeyer
:jdwp=transport=dt_socket,server=y,suspend=y,address=3' On Fri, Jan 13, 2017 at 12:06 PM, John Blum wrote: Yes, this works. I have done this before. Of course, you need to properly coordinate between the debugger and the server starting up. On Fri, Jan 13, 2017 at 11:54 AM, Udo Kohlme

Re: Native Client Directory Structure

2017-01-16 Thread Udo Kohlmeyer
+1 There are discussions relating to the pure Java client. I think it would fight right into the "geode-client" project. --Udo On 1/16/17 08:26, Jacob Barrett wrote: One of the first things necessary to get NC merged into the the develop branch is understanding where it will go under the cu

Re: Native Client Directory Structure

2017-01-16 Thread Udo Kohlmeyer
-1 "geode-native" directory name +1 "geode-client" directory name Maybe the directories for the different clients are by language, so we omit the "geode" prefix i.e geode-client/ c++, net java python If clients are in their own project, then the clients can be independen

Re: Native Client Directory Structure

2017-01-16 Thread Udo Kohlmeyer
sitory? Now that we are a TLP, we do have that capability, as seen with the 'geode-examples' repository. --Mark On Mon, Jan 16, 2017 at 10:38 AM, Udo Kohlmeyer wrote:

Re: Review Request 55644: GEODE-2277 client cache fails to deserialize a PdxInstance due to InternalGemFireError

2017-01-17 Thread Udo Kohlmeyer
GatewayReceiverCommand.cmdExecute method... Why would we not have refactored at 700 line long method... made it really hard to follow the flow of the fix. Maybe we need to open a ticket for this refactor. - Udo Kohlmeyer On Jan. 17, 2017, 9:23 p.m., Hitesh Khamesra wrote

Review Request 55652: GEODE-2003: REST Function invocation fails on fully qualified Function names

2017-01-17 Thread Udo Kohlmeyer
nning Thanks, Udo Kohlmeyer

Re: Addressing OOM while recovering overflow regions.

2017-01-17 Thread Udo Kohlmeyer
I think that #1 is a good start... I think we should have all regions created before we recover from the files, or at least for the regions that use the diskstore being recovered. When we recover, do we start recovering in reverse? I mean, the latest data *should* be in the last data files, s

<    1   2   3   4   5   6   >