Re: CODEOWNERS? (was Re: Pending PR reviews)

2022-06-29 Thread Jinmei Liao
+1 to Anthony's suggestion From: Donal Evans Date: Wednesday, June 29, 2022 at 10:46 AM To: dev@geode.apache.org Subject: Re: CODEOWNERS? (was Re: Pending PR reviews) ⚠ External Email +1 to Anthony's suggestion I strongly supported the idea behind CODEOWNERS when it was originally

Re: [PROPOSAL] RFC for migrating from springfox to springdoc

2022-05-05 Thread Jinmei Liao
>Does this inactivity demonstrate maturity (i.e. maybe we should remove the >@Experimental annotation) or does it indicate a failed or abandoned experiment Neither, I believe k8s is using the feature, but the inactivity simply means we haven’t allocated people to work on it. From: Owen Nichols

Re: [VOTE] Apache Geode 1.14.2.RC1

2021-12-15 Thread Jinmei Liao
+1 From: Nabarun Nag Date: Wednesday, December 15, 2021 at 11:00 AM To: dev@geode.apache.org Subject: Re: [VOTE] Apache Geode 1.14.2.RC1 +1 to 1.14.2.RC1 * Build from source * WAN clusters with SSL * puts/gets * validate data integrity after rolling restart. Regards Nabarun

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

2021-12-15 Thread Jinmei Liao
+1 From: Nabarun Nag Date: Wednesday, December 15, 2021 at 10:34 AM To: dev@geode.apache.org Subject: Re: [Suspected Spam] [VOTE] Apache Geode 1.13.6.RC1 +1 to 1.13.6.RC1 * Build from source * WAN clusters with SSL * puts/gets * validate data integrity after rolling restart.

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

2021-12-15 Thread Jinmei Liao
+1 From: Nabarun Nag Date: Wednesday, December 15, 2021 at 10:11 AM To: dev@geode.apache.org Subject: Re: [Suspected Spam] [VOTE] Apache Geode 1.12.7.RC1 +1 to 1.12.7.RC1 * Build from source * WAN clusters with SSL * puts/gets * validate data integrity after rolling restart.

Re: @TestOnly or @VisibleForTesting

2021-11-04 Thread Jinmei Liao
My understanding is @VisibileForTesting methods are used by the products, while @TestOnly methods are used only by the tests. In practice, I don’t like to add @TestOnly methods (although I like to mark those methods with this annotation if I found out a method is only used for testing for

Re: [discuss] RFC for Geode Authentication Expiation and Re-Authentication

2021-08-23 Thread Jinmei Liao
the "register interests and CQ"? Is that by unregistering or queueing? I think this RFC looks good. Thanks, Mark On 7/27/21, 2:26 PM, "Jinmei Liao" wrote: Calling more feedback on this RFC. I will move this to “Under Development” if no objection to its general direction

Re: Annual branch cleanup Aug 17

2021-08-17 Thread Jinmei Liao
Please leave the feature branch “expireAuthentication”, The RFC is in: https://cwiki.apache.org/confluence/display/GEODE/On+Demand+Geode+Authentication+Expiration+and+Re-authentication From: Owen Nichols Date: Sunday, August 1, 2021 at 1:01 AM To: dev@geode.apache.org Subject: Annual branch

Re: [discuss] RFC for Geode Authentication Expiation and Re-Authentication

2021-07-27 Thread Jinmei Liao
Calling more feedback on this RFC. I will move this to “Under Development” if no objection to its general direction end of this Thursday. Thanks! From: Jinmei Liao Date: Thursday, July 22, 2021 at 5:37 PM To: dev@geode.apache.org Subject: [discus] RFC for Geode Authentication Expiation and Re

[discus] RFC for Geode Authentication Expiation and Re-Authentication

2021-07-22 Thread Jinmei Liao
Hi, Fellow devs, Here the feature proposal for the said topic. Please review and provide your feedback. Thanks! https://cwiki.apache.org/confluence/display/GEODE/On+Demand+Geode+Authentication+Expiration+and+Re-authentication Jinmei

Re: How to add a new REST end point in Cluster Management Service

2020-11-10 Thread Jinmei Liao
itten more like a how-to. So given the feature is still in progress. Do you want feedback on the writing of the document or the architecture of the underlying code or both? From: Jinmei Liao Sent: Tuesday, November 10, 2020 10:06 To: dev@geode.

Re: How to add a new REST end point in Cluster Management Service

2020-11-10 Thread Jinmei Liao
, John From: Jinmei Liao Sent: Monday, October 26, 2020 10:53 AM To: dev@geode.apache.org Subject: How to add a new REST end point in Cluster Management Service Hi, Geode developers, I just added a twiki page detailing how to add a new REST

Re: [PROPOSAL] backport GEODE-8651 to 1.13, 9.10, 9.9

2020-10-27 Thread Jinmei Liao
+1 On Oct 27, 2020 3:00 PM, Donal Evans wrote: +1 From: Anthony Baker Sent: Tuesday, October 27, 2020 2:53 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] backport GEODE-8651 to 1.13, 9.10, 9.9 +1 from me > On Oct 27, 2020, at 2:21 PM, Xiaojian Zhou

How to add a new REST end point in Cluster Management Service

2020-10-26 Thread Jinmei Liao
Hi, Geode developers, I just added a twiki page detailing how to add a new REST end point in Cluster Management Service to manage new entities or start new long running operations in Geode Cluster. Feel free to look it through and provide comments/suggestions.

[PROPOSAL] backport fix for GEODE-8574 to 1.13.1

2020-10-08 Thread Jinmei Liao
I would like to include the fix for GEODE-8574 to 1.13.1, it would greatly help the Geode on k8s experience. Thanks! Jinmei

Re: [DISCUSS] Supported filename convention for Deploy Jars functionality

2020-10-07 Thread Jinmei Liao
we even need a classification scheme? IOW, why not just take what the user gives us and say thank you :-). Is this restriction imposed by our *implementation* choices? Anthony > On Oct 7, 2020, at 3:24 PM, Jinmei Liao wrote: > > Wait, t

Re: [DISCUSS] Supported filename convention for Deploy Jars functionality

2020-10-07 Thread Jinmei Liao
Wait, that reason doesn't make much sense either. Dale/Darrel, do you remember why we did what we did? On 10/7/20, 3:12 PM, "Jinmei Liao" wrote: I believe we did this for a reason, can't remember exactly what though. Most probably drive by user's existing filenames. I

Re: [DISCUSS] Supported filename convention for Deploy Jars functionality

2020-10-07 Thread Jinmei Liao
I believe we did this for a reason, can't remember exactly what though. Most probably drive by user's existing filenames. I believe we are probably concerned that user's jar name might contain "_" or "-" themselves, like common-logging.jar etc. So we had to resort to finding the first "."

Re: [VOTE] Apache Geode 1.13.0.RC1

2020-09-08 Thread Jinmei Liao
+1 Verified geode management rest urls are working as expected. On 9/8/20, 9:53 AM, "Owen Nichols" wrote: +1 I have reviewed test results for a battery of internal functional and performance tests, reviewed the LICENSE file for accuracy, looked at the logs for all checks in the rc

Re: [PROPOSAL] Backport GEODE-8432 to 1.13

2020-08-19 Thread Jinmei Liao
+1 On 8/19/20, 10:25 AM, "Xiaojian Zhou" wrote: This problem also exists in 1.13.

Re: Proposal to backport GEODE-8395 (gfsh help banner) to support branches

2020-08-02 Thread Jinmei Liao
+1 On Aug 1, 2020 10:30 PM, Owen Nichols wrote: This issue was present since Geode 1.0. There is zero risk from this fix, which is already on develop. It is critical because Geode releases are a legal Act of the Apache Foundation and should not misrepresent the identity of the project.

Re: [Proposal] Back-port doc fixes to support/1.13

2020-07-22 Thread Jinmei Liao
+1 On Jul 22, 2020 3:39 PM, Dave Barnes wrote: I propose back-porting the following doc updates to Geode support/1.13 (and 1.12, while we're at it): - GEODE-2113: User Guide - p2p.HANDSHAKE_POOL_SIZE is obsolete, remove from docs (code fixed in 1.9.0, docs fixed in 1.14.0) - GEODE-7628: Block

[PROPOSAL] Cherry pic GEODE-8331 to support branches

2020-07-22 Thread Jinmei Liao
I would like to propose to cherry pick GEODE-8331: allow GFSH to connect to other versions of cluster (#5375) to support branches up to 1.10. This would allow gfsh to connect to other versions of cluster and provide better error messages when command is not support by the connected cluster.

Re: [PROPOSAL] merge GEODE-8259 to support branches

2020-06-30 Thread Jinmei Liao
+1 From: Owen Nichols Sent: Tuesday, June 30, 2020 3:56 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] merge GEODE-8259 to support branches +1 On 6/30/20, 3:51 PM, "Xiaojian Zhou" wrote: Customer encountered a singlehop getAll failure due to

Re: Us vs Docker vs Gradle vs JUnit

2020-06-30 Thread Jinmei Liao
I would vote for fixing the tests to use gradle's normal forking. If we are going to invest time and effort, let's invest in an option that can reduce our dependencies From: Jacob Barrett Sent: Tuesday, June 30, 2020 11:30 AM To: dev@geode.apache.org Subject:

Re: [Proposal] Back-port doc fixes to support/1.13

2020-06-29 Thread Jinmei Liao
+1, yay to doc changes. From: Dave Barnes Sent: Monday, June 29, 2020 10:30 AM To: dev@geode.apache.org Subject: [Proposal] Back-port doc fixes to support/1.13 These five doc changes correct doc bugs and format errors that have caused users pain. I propose that

Re: [PROPOSAL] Add windows jobs to PR checks

2020-06-25 Thread Jinmei Liao
+1, what was the reason for it not being included the PR before? From: Dick Cavender Sent: Thursday, June 25, 2020 9:54 AM To: dev@geode.apache.org Subject: RE: [PROPOSAL] Add windows jobs to PR checks +1 -Original Message- From: Owen Nichols Sent:

Re: Odg: Certificate Based Authorization

2020-06-19 Thread Jinmei Liao
In the old management team, we have been considering the idea of getting rid of jmx connection in gfsh and only using http connection mechanism. On Jun 19, 2020 2:53 PM, Jacob Barrett wrote: So I can see why this research paper was so bleak about the options in trying to get the SSL

Re: [PROPOSAL] back port fix for GEODE-8251 to support branches

2020-06-19 Thread Jinmei Liao
Need one more vote  From: Bruce Schuchardt Sent: Thursday, June 18, 2020 3:43 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] back port fix for GEODE-8251 to support branches +1 On 6/18/20, 3:24 PM, "Jinmei Liao" wrote: The fix for this is

[PROPOSAL] back port fix for GEODE-8251 to support branches

2020-06-18 Thread Jinmei Liao
The fix for this issue https://issues.apache.org/jira/browse/GEODE-8251 is needed on support 1.13 branch in order for rolling upgrade to work from 1.12 to 1.13. Thanks! Jinmei

Re: Refactor to Restore Redundancy Command for 1.13?

2020-06-15 Thread Jinmei Liao
+1 for making it simpler for upgrade. From: Owen Nichols Sent: Monday, June 15, 2020 1:41 PM To: dev@geode.apache.org Subject: Re: Refactor to Restore Redundancy Command for 1.13? There is precedent[1] for bringing a refactoring to a support branch prior to

Re: [DISCUSSION] Stop using the Geode Repository for Feature/WIP Branches

2020-06-11 Thread Jinmei Liao
+1 for ban the use of geode repo for feature branches. Less clutter is better. From: Jacob Barrett Sent: Tuesday, June 2, 2020 3:42 PM To: dev@geode.apache.org Subject: [DISCUSSION] Stop using the Geode Repository for Feature/WIP Branches I know this has been

Re: [PROPOSAL] make Cluster Management Service CRUD operations thread safe

2020-05-28 Thread Jinmei Liao
quest message to the grantor. The cost is with sending message to the Grantor, in most cases. Which is not bad, considering the configuration does not change frequently. -Anil. On 5/28/20, 11:08 AM, "Jinmei Liao" wrote: Simultaneous updates to configurations are already protected by a diffe

Re: [PROPOSAL] make Cluster Management Service CRUD operations thread safe

2020-05-28 Thread Jinmei Liao
ter a period of time? Anthony > On May 28, 2020, at 10:17 AM, Jinmei Liao wrote: > > The proposal is proposing using ONE single dlock to synchronize all CMS CRUD > operations, that means, in a given time, only one CRUD operation in CMS is > allowed in the entire cluster, this see

Re: [PROPOSAL] make Cluster Management Service CRUD operations thread safe

2020-05-28 Thread Jinmei Liao
ID element. That way, we allow more concurrency. I am just not sure what's the cost of creating a dlock. Is the the cost of creating a dlock per ID warrants the performance gain? Comment/Suggestions? Jinmei From: Jinmei Liao Sent: Tuesday, May 26, 2020 1:02 PM

[PROPOSAL] make Cluster Management Service CRUD operations thread safe

2020-05-28 Thread Jinmei Liao
Hi, Geode Community, Currently, the CMS CRUD operations are not thread safe, if one call tries to create a region, and another call tries to delete the same region, if timing is off, we could end up with inconsistent state (what's in cluster config and what's actually on the server). So we

Re: [PROPOSAL] backport GEODE-8174 to 1.13 and 1.12

2020-05-27 Thread Jinmei Liao
+1 From: Owen Nichols Sent: Tuesday, May 26, 2020 4:09 PM To: dev@geode.apache.org Subject: Re: [PROPOSAL] backport GEODE-8174 to 1.13 and 1.12 +1 On 5/26/20, 4:04 PM, "Eric Shu" wrote: +1 From: Udo Kohlmeyer

Re: [PROPOSAL] Bring GEODE-8100 to support branches

2020-05-20 Thread Jinmei Liao
branch, but would like an opportunity to edit for a few > > > proof-readerish items that caught my eye. Will submit a PR today for > your > > > review, Jinmei. Thanks for your contribution! > > > > > > > > On 2020/05/20 14:09:05, Joris Melchior wrote: >

Re: [PROPOSAL] bring GEODE-8131 PR to support branches

2020-05-19 Thread Jinmei Liao
+1 On Tue, May 19, 2020 at 10:05 AM Udo Kohlmeyer wrote: > +1 > On May 19, 2020, 8:53 AM -0700, Bruce Schuchardt , > wrote: > While investigating a distributed hang we discovered that the alerting > system was blocking the logging of critical information that would have > helped diagnose the

[PROPOSAL] Bring GEODE-8100 to support branches

2020-05-19 Thread Jinmei Liao
This is to update documentation to better explain the Cluster management service and various geode/system properties that control the behavior of it. It also provides more usage examples in the documentation. There is no product code change in this, but tt would be helpful to the users who would

[PROPOSAL] bring GEODE-8091 to support branches

2020-05-11 Thread Jinmei Liao
https://issues.apache.org/jira/browse/GEODE-8091 We've had users that were trying to use the "--load-cluster-configuration-from-dir=true" when starting up a locator with a security manager and came across this failure on Geode1.12 and would this to be fixed. Can I get a few +1s to port this back

Re: Over usage of @SuppressWarnings

2020-05-08 Thread Jinmei Liao
It's not the test code, it's the "start locator" command itself. On Fri, May 8, 2020 at 2:27 PM Jacob Barrett wrote: > > > On May 8, 2020, at 1:55 PM, Jinmei Liao wrote: > > > > What's the recommendation for the legitimate usage of the deprecated > > pro

Re: Over usage of @SuppressWarnings

2020-05-08 Thread Jinmei Liao
What's the recommendation for the legitimate usage of the deprecated property? In my case, a gemFire property is deprecated, but "start locator" command still has an option to turn on that property (the option is deprecated as well, but we are still obligated to keep it in the code). In this case,

Re: [PROPOSAL]: GEODE-8071 to support/1.12 and support/1.13

2020-05-07 Thread Jinmei Liao
+1 On Thu, May 7, 2020 at 9:26 AM Donal Evans wrote: > +1 > > From: Dick Cavender > Sent: Thursday, May 7, 2020 8:52 AM > To: dev@geode.apache.org > Subject: Re: [PROPOSAL]: GEODE-8071 to support/1.12 and support/1.13 > > +1 > > On Thu, May 7, 2020 at 7:47 AM

Re: [PROPOSAL] include GEODE-8073 in Geode 1.13 support branch

2020-05-06 Thread Jinmei Liao
+1 On Wed, May 6, 2020 at 11:40 AM Owen Nichols wrote: > +1 to fix this NPE on support/1.13 and also support/1.12 > > > On May 6, 2020, at 11:19 AM, Eric Shu wrote: > > > > GEODE-8073 > > -- Cheers Jinmei

[PROPOSAL] include GEODE-8055 in Geode 1.12 support branch

2020-05-06 Thread Jinmei Liao
I would like to include the fix for GEODE-8055 in the 1.12 support branch. This would allow users to use gfsh to create an index on sub regions. -- Cheers Jinmei

Re: [PROPOSAL] include GEODE-8055 in support/1.13

2020-05-05 Thread Jinmei Liao
t; > > was sent. +1 from me to go ahead and bring to support/1.13 on that > > basis. > > > > > > > On May 4, 2020, at 7:43 PM, Jinmei Liao wrote: > > > > > > > > Yes, there is a user request to reinstate this feature. > > > > > > > > O

Re: [PROPOSAL] include GEODE-8055 in support/1.13

2020-05-04 Thread Jinmei Liao
nt to get this in 1.13. > > -Anil. > > > On Mon, May 4, 2020 at 11:55 AM Jinmei Liao wrote: > > > I would like to include the fix for GEODE-8055 in the 1.13 branch. This > > would allow users to use gfsh to create an index on sub regions. > > > > -- > > Cheers > > > > Jinmei > > > -- Cheers Jinmei

[PROPOSAL] include GEODE-8055 in support/1.13

2020-05-04 Thread Jinmei Liao
I would like to include the fix for GEODE-8055 in the 1.13 branch. This would allow users to use gfsh to create an index on sub regions. -- Cheers Jinmei

Re: GemFireCacheImplCloseTest flakiness

2020-05-01 Thread Jinmei Liao
It failed on my PR as well. On Fri, May 1, 2020 at 9:23 AM Kirk Lund wrote: > I don't see anything obvious in the test so I’m going to mark the test > @Ignore until I have a fix. Please review: > https://github.com/apache/geode/pull/5038 > > On Fri, May 1, 2020 at 9:00 AM Kirk Lund wrote: > >

Re: [DISCUSS] etiquette around breaking the pipeline

2020-04-29 Thread Jinmei Liao
1. create the revert PR ASAP 2. work on the fix properly and create the fix PR 3. wait and merge whichever goes green and approved first. On Wed, Apr 29, 2020 at 7:13 PM Joris Melchior wrote: > Recent experience makes me lean towards quick revert as well. Takes the > pressure off when trying to

Re: Proposal to restore Pulse logging in support/1.12

2020-04-24 Thread Jinmei Liao
+1 On Fri, Apr 24, 2020 at 2:53 PM Anthony Baker wrote: > +1 > > > On Apr 24, 2020, at 2:46 PM, Dale Emery wrote: > > > > During the cleanup of the gradle build and logging, the Pulse webapp > lost its slf4j implementation. As a result, Pulse stopped writing log files. > > > > I’ve restored

Re: Proposal to bring GEODE-7941 to support/1.12

2020-04-06 Thread Jinmei Liao
+1 to backport On 4/6/20, 9:14 AM, "Anthony Baker" wrote: +1 to backport > On Apr 6, 2020, at 8:54 AM, Owen Nichols wrote: > > Recently some Geode users have expressed concern that shiro-1.4.1.jar is getting flagged for critical security vulnerability CVE-2020-1957.

Re: [DISCUSS] Redundancy Gfsh Commands

2020-03-30 Thread Jinmei Liao
So the "restore redundancy" command is blocking and only returns when the operation is finished? On 3/30/20, 2:21 PM, "Kirk Lund" wrote: [I added this as a comment on the wiki page] You might want to consider making RestoreRedundancyOperation actually extend

[DISCUSS] RFC: How management rest API GET request deals with entities in multiple groups

2020-03-05 Thread Jinmei Liao
Hi everyone, I would like to summarize what the management REST GET request would return if the entity exists in multiple groups. The linked RFC[1] has all the details. [1] https://cwiki.apache.org/confluence/display/GEODE/Group+Configuration+in+Management+Rest+API -- Cheers Jinmei

Re: Help needed to get my PR to pass the stressNewTest

2020-03-04 Thread Jinmei Liao
Have you tried running all the changed tests in your IDEA repeatedly to see if there would be a failure? On Wed, Mar 4, 2020 at 1:28 PM Eric Shu wrote: > My PR (https://github.com/apache/geode/pull/4709) continue to fail in > stressNewTest. I have been retrigger the test and all failed with

Re: [PROPOSAL]: Include GEODE-7756 in Release 1.12.0

2020-02-13 Thread Jinmei Liao
+1 On Thu, Feb 13, 2020, 6:47 AM Owen Nichols wrote: > +1 > > On Thu, Feb 13, 2020 at 3:38 AM Ju@N wrote: > > > Hello devs, > > > > I'd like to include the fix for GEODE-7756 [1] in release 1.12.0. > > The change prevents a performance degradation introduced in Geode 1.11 > > through to the

Re: [PROPOSAL]: Include GEODE-7789 in Release 1.12.0

2020-02-13 Thread Jinmei Liao
+1 On Thu, Feb 13, 2020, 6:47 AM Owen Nichols wrote: > +1 > > On Thu, Feb 13, 2020 at 3:05 AM Nabarun Nag wrote: > > > +1 > > > > On Thu, Feb 13, 2020 at 2:09 AM Ju@N wrote: > > > > > Hello devs, > > > > > > I'd like to include the fix for GEODE-7789 [1] in release 1.12.0. > > > The change

Re: Include GEODE-7776 in release 1.12

2020-02-11 Thread Jinmei Liao
+1 On Tue, Feb 11, 2020, 11:39 AM Udo Kohlmeyer wrote: > +1 > > On 2/11/20 11:23 AM, Dick Cavender wrote: > > This regression was introduced when the geode-gfsh subproject was > recently > > added. While not obvious this created a critical build / runtime cycle > > between geode-core and

Re: [PROPOSAL]: Include GEODE-7717 in Release 1.12.0

2020-02-07 Thread Jinmei Liao
+1 On Fri, Feb 7, 2020 at 8:58 AM Bruce Schuchardt wrote: > +1 > > On 2/7/20, 8:36 AM, "Jinmei Liao" wrote: > > Hi devs, > > I would like to include the fix for GEODE-7717[1] in release 1.12.0. > The > change modifies the json response struct

Re: please include the fix for geode-7750 and geode-7760 in 1.12

2020-02-07 Thread Jinmei Liao
+1 On Fri, Feb 7, 2020 at 9:06 AM Owen Nichols wrote: > +1 > > On Fri, Feb 7, 2020 at 9:04 AM Udo Kohlmeyer wrote: > > > +1 > > > > On 2/7/20 8:57 AM, Bruce Schuchardt wrote: > > > af8307 fixes a number of problems with auto-reconnect that have been > > reported recently. Owen and Darrel did

[PROPOSAL]: Include GEODE-7717 in Release 1.12.0

2020-02-07 Thread Jinmei Liao
Hi devs, I would like to include the fix for GEODE-7717[1] in release 1.12.0. The change modifies the json response structure of all of the "list" operations in cluster management rest service. Including it in this release, where most users are just starting to experiment with CMS rest service

Re: [Vote] Include GEODE-7752 into 1.12

2020-02-05 Thread Jinmei Liao
Even though the management service is still marketed as "experimental" and we don't need to go through deprecations, I would still very much prefer this gets pulled into 1.12 so that users don't have to deal with experimental public API changes. On Wed, Feb 5, 2020 at 4:23 PM Udo Kohlmeyer

Re: [Vote] Include GEODE-7752 into 1.12

2020-02-05 Thread Jinmei Liao
+1 On Wed, Feb 5, 2020 at 2:36 PM John Blum wrote: > +1 > > On Wed, Feb 5, 2020 at 1:54 PM Patrick Johnson > > wrote: > > > +1 > > > > On 2/5/20, 1:53 PM, "Udo Kohlmeyer" wrote: > > > > Hi there Geode dev, > > > > I would like to request that GEODE-7752 > >

Re: Release branch 1.12.0 created

2020-02-03 Thread Jinmei Liao
I just reverted a commit that I made last Friday, Please also include this commit "16b296a4" in the release branch. Thanks! On Mon, Feb 3, 2020 at 11:07 AM Ernest Burghardt wrote: > Hello Geode Dev Community, > > We have created a new release branch for Apache Geode 1.12.0 - > "release/1.12.0"

Re: startClusterManagementService throws NPE

2020-01-10 Thread Jinmei Liao
Is there a specific test that would cause this NPE? I will try to take a look at it. On Fri, Jan 10, 2020 at 9:08 AM Kirk Lund wrote: > I think this may only occur in tests that perform a reconnect of the > Locator. > > On Fri, Jan 10, 2020 at 9:06 AM Kirk Lund wrote: > > > Anyone know why we

Re: RFC - Logging to Standard Out

2020-01-08 Thread Jinmei Liao
+1 On Wed, Jan 8, 2020 at 7:31 PM Robert Houghton wrote: > +1 > > On Wed, Jan 8, 2020, 12:39 Jacob Barrett wrote: > > > Please see RFC for Logging to Standard Out. > > > > > https://cwiki.apache.org/confluence/display/GEODE/Logging+to+Standard+Out > > < >

Re: [DISCUSS] abandon branch protection rules

2019-12-30 Thread Jinmei Liao
Having the stressTest reuse the JVMs is close to running the tests in my IDEA repeatedly for N times or running a package of tests together in my IDEA. There was a time that I couldn't run a group of tests together in my IDEA until I had to fix the problem for the stressTest. Keeping them running

Re: [DISCUSS] `status locator` command fail when locator's ssl is turned on

2019-12-30 Thread Jinmei Liao
he term "usability" because > > users expect "status server --host=xxx --port=xxx" to also work IF we > > continue to provide "status locator --host=xxx --port=xxx". We need to > get > > rid of and avoid these weird quirks and inconsistencies. > >

Re: [DISCUSS] `status locator` command fail when locator's ssl is turned on

2019-12-23 Thread Jinmei Liao
on-SSL-enabled client to an SSL-enabled locator. > > > > > > > > > It would seem very weird if I have to provide additional connection > > params > > > to the 'status' command if I've already provided them as part of the > > > 'connect'. Could we not st

Re: [DISCUSS] Proposal to require linear commit history on develop

2019-12-20 Thread Jinmei Liao
tests now pass > > >>>>> reliably but I was very frustrated by the process. > > >>>>>> On 12/19/19 4:49 PM, Jacob Barrett wrote: > > >>>>>>> I’m in agreement with Dan. Changes to the infrastructure to flat >

Re: [DISCUSS] Proposal to require linear commit history on develop

2019-12-20 Thread Jinmei Liao
lanket restriction. In fact I think our > release > > >> process involves some merges. > > >> > > >> I think setting standards on what is reasonable to be an individual > > commit > > >> will do a lot more to clean up our history than blocking

Re: [DISCUSS] `status locator` command fail when locator's ssl is turned on

2019-12-19 Thread Jinmei Liao
uent usage? > > --Jens > > On Wed, Dec 18, 2019 at 9:04 AM Jinmei Liao wrote: > > > "status locator" command is broken on ssl enabled locators ever since we > > fixed a bug that leaked the connection properties from one tcp socket > > connection to

Re: [DISCUSS] Proposal to require linear commit history on develop

2019-12-19 Thread Jinmei Liao
+1 On Thu, Dec 19, 2019, 4:05 PM Owen Nichols wrote: > I’d like to advance this topic from an informal request/discussion to a > discussion of a concrete proposal. > > To recap, it sounds like there is general agreement that commit history on > develop should be linear (no merge commits), and

[DISCUSS] `status locator` command fail when locator's ssl is turned on

2019-12-18 Thread Jinmei Liao
"status locator" command is broken on ssl enabled locators ever since we fixed a bug that leaked the connection properties from one tcp socket connection to another. Before that it would just magically work if we have previously made a successfully tcp connection to that same locator, now we are

Re: [DISCUSS] - Move gfsh into its own submodule

2019-11-22 Thread Jinmei Liao
+100. Would be a great move. On Fri, Nov 22, 2019 at 8:40 AM Jens Deppe wrote: > Hello All, > > We'd like to propose moving gfsh and all associated commands into its own > gradle submodule (implicitly thus also producing a separate maven > artifact). The underlying intent is to decouple geode

Re: [DISCUSS] is overriding a PR check ever justified?

2019-10-31 Thread Jinmei Liao
; same question. [the process won't take more than 30 min, also its good to > confirm that the revert won't turn the pipeline red] > I am more worried that how a commit that made the pipeline red made it into > the codebase. > > Regards > Naba > > On Thu, Oct 31, 2019 at 2:37 PM

Re: [vote/discuss]Override stressNewTest for Pull Request #4250?

2019-10-31 Thread Jinmei Liao
+1 On Thu, Oct 31, 2019, 3:30 PM Xiaojian Zhou wrote: > I'm curious to see the new stressNew test result too. > > On Thu, Oct 31, 2019 at 3:26 PM Owen Nichols wrote: > > > I’ve retriggered StressNew < > > >

Re: [DISCUSS] is overriding a PR check ever justified?

2019-10-31 Thread Jinmei Liao
I am not sure whether this is related to this topic or not, but recently I had to revert one of my commit, before I could just do a revert and push to develop, but now that is blocked. I had to file a PR to revert a change that's causing the pipeline to be red. My question is: do we still need to

Re: [DISCUSS] Tweak to branch protection rules

2019-10-29 Thread Jinmei Liao
+1 On Tue, Oct 29, 2019, 6:08 PM Owen Nichols wrote: > +1 …this has already bitten me a few times > > > On Oct 29, 2019, at 6:01 PM, Dan Smith wrote: > > > > Hi all, > > > > It seems we've configured our branch protection rules such that pushing a > > change to a PR that has been approved

Re: [VOTE] Apache Geode 1.9.2.RC1

2019-10-22 Thread Jinmei Liao
+1 On Tue, Oct 22, 2019 at 11:47 AM Dave Barnes wrote: > +1 > Downloaded, successfully built Geode and Geode-Native docs form source. > > On Tue, Oct 22, 2019 at 2:17 AM Ju@N wrote: > > > +1, > > > > Downloaded and built from source, created two clusters with multiple > > members and verified

Re: Token based authentication support added in Geode Develop

2019-10-04 Thread Jinmei Liao
ity to recognize a HTTP > authentication header containing 'Bearer ' and > then handing that to the Security Manager to do with as it pleases? > > We're not doing anything with actual token management? (i.e. generating, > revoking, etc.). > > --Jens > > On Fri, Oct 4, 2019 at 10:59 A

Token based authentication support added in Geode Develop

2019-10-04 Thread Jinmei Liao
Hi, all JWT token based authentication support is added to Geode develop branch. Currently only management v2 rest api can use this (we can add dev rest there too if requested). In order to turn on token based auth for management rest api, you will need to do these two things: 1. start your

Re: [PROPOSAL] adding java-jq to GEODE dependency for testing

2019-09-25 Thread Jinmei Liao
> > On Wed, Sep 25, 2019 at 8:39 AM Anthony Baker wrote: > > > >> Sounds good, thanks for the heads up. > >> > >> Anthony > >> > >> > >>> On Sep 25, 2019, at 8:37 AM, Jinmei Liao wrote: > >>> > >>> Mana

Re: [VOTE] Apache Geode 1.10.0.RC2

2019-09-23 Thread Jinmei Liao
+1 verified the management v2 apis are available. On Mon, Sep 23, 2019 at 12:58 PM Anthony Baker wrote: > +1 > > > Reviewed: > > - Signatures and hashes > - LICENSE and NOTICE > - No binaries in source distribution > - Builds from source > > > Quibbles: > > - Let’s include geode-benchmarks/

Re: [VOTE] Apache Geode 1.10.0.RC1

2019-08-30 Thread Jinmei Liao
+ 1 verified the management v2 api is available by default and list of rest end points. On Fri, Aug 30, 2019 at 2:06 PM Dick Cavender wrote: > Hello Geode dev community, > > This is a release candidate for Apache Geode, version 1.10.0.RC1. > Thanks to all the community members for their

Re: [DISCUSS] what region types to support in the new management rest api

2019-08-20 Thread Jinmei Liao
:07 AM, Alexander Murmann > > > > wrote: > > > > > > > > Hey folks, I want to make sure that any other's product's roadmaps > have > > > no > > > > impact on any decisions we make about Apache Geode. > > > > > > > > Thanks! > > &g

Re: [DISCUSS] what region types to support in the new management rest api

2019-08-20 Thread Jinmei Liao
>> > >>> > >>>> On Tue, Aug 20, 2019 at 12:10 PM Kirk Lund wrote: > >>>> > >>>> Here's my 2cents: The Geode Management REST API should definitely > >> support > >>>> "group" such that creation of a

Re: [DISCUSS] what region types to support in the new management rest api

2019-08-20 Thread Jinmei Liao
; > > these regions would be created on servers. > > > So, for example, does anyone see a need to create PROXY regions on the > > > server? Even if we did not support them on the server, they would still > > be > > > supported on clients. > > > &

[DISCUSS] what region types to support in the new management rest api

2019-08-19 Thread Jinmei Liao
Region type (in another word Region shortcut) defines a set of attributes for a region. These are the list of region types we have: LOCAL, LOCAL_PERSISTENT, LOCAL_HEAP_LRU, LOCAL_OVERFLOW, LOCAL_PERSISTENT_OVERFLOW, PARTITION, PARTITION_REDUNDANT, PARTITION_PERSISTENT,

Re: [DISCUSS] Time to cut Geode 1.10.0?

2019-07-22 Thread Jinmei Liao
I believe it's realistic to assume that On Mon, Jul 22, 2019, 3:58 PM Alexander Murmann wrote: > Jinmei, > > Do you think it's realistic to add the property this week and still cut the > branch at the end of this week? > > On Mon, Jul 22, 2019 at 3:38 PM Jinmei Liao wrote: &

Re: [DISCUSS] Time to cut Geode 1.10.0?

2019-07-22 Thread Jinmei Liao
Work is still being planned to move the cluster management rest service under an experimental version flag and use a geode property to turn it on/off. I would say we are ready to cut the geode 1.10.0 after that work is complete. On Mon, Jul 22, 2019 at 3:24 PM Alexander Murmann wrote: > Hi

Re: Naming System for Regions - Active Jira Question

2019-07-15 Thread Jinmei Liao
I added the comment on this issue: Geode region names ARE case sensitive. Should not fix. On Mon, Jul 15, 2019 at 10:51 AM Alex Grisham wrote: > Hello everyone, > > I’ve recently gotten into the Geode community and have been looking for a > good place to start contributing. I found this Jira

Re: [DISCUSS]: ClusterManagementService configuration objects

2019-07-11 Thread Jinmei Liao
They will be public API, currently, mainly used by the ClusterManagementServivice to configure the cluster. On Wed, Jul 10, 2019 at 9:06 PM Jacob Barrett wrote: > Are these new objects public API or internal? > > > On Jul 10, 2019, at 1:16 PM, Jinmei Liao wrote: > > >

[DISCUSS]: ClusterManagementService configuration objects

2019-07-10 Thread Jinmei Liao
We've been working on a new and improved ClusterManagmentService for a while now. It allows developers/administrators to manage the clusters through rest calls instead of having to use gfsh (more info here: https://cwiki.apache.org/confluence/display/GEODE/Cluster+Management+Service ). Up until

Re: Unnecessary uses of final on local variables

2019-06-18 Thread Jinmei Liao
I agree with Murtuza, most finals on local variables and method parameters are just noise to me. I only use "final" on these two situations: 1. to declare public static constants of immutable types (e.g. String, Integer) 2. to prevent children from overriding a method. But thought I can't offer

Re: [PROPOSAL] Add hostname-for-clients to ConfigurationProperties

2019-06-04 Thread Jinmei Liao
ing those settings as well, or just having a global > > property that is the default value if it is not overridden for a specific > > locator, cache-server, or gateway-receiver? > > > > -Dan > > > > On Tue, Jun 4, 2019 at 12:02 PM Jinmei Liao wrote: > > > &

[PROPOSAL] Add hostname-for-clients to ConfigurationProperties

2019-06-04 Thread Jinmei Liao
We have "jmx-manager-hostname-for-clients" in the geode properties, we think it would be a good idea to deprecate that and use "hostname-for-clients" for the entire server. Currently we already need this property when launching a locator and start a gateway-receiver, and we have no way to

Re: what is the best way to update a geode pull request

2019-05-31 Thread Jinmei Liao
+1 for initial PR needs to be a single commit +1 for making intermittent commits to the PR to show history of revision. I would only do a rebase and force-push if github tells me that there is a conflict, but my rebase would keep my original commits (no squashing). I do not like to do a merge,

Re: [DISCUSS] require reviews before merging a PR

2019-05-31 Thread Jinmei Liao
I was told that screenshot that I sent earlier got filtered out by the dev list. Basically, the filter puts "notificati...@github.com" in the "From" section, and put "review_reques...@noreply.github.com" in the "Doesn't have" section of the form. On Fri, May 31, 2019 at 10:36 AM Anthony Baker

  1   2   3   4   5   6   7   >