Re: Geode 1.9 Release Manager

2019-02-14 Thread Jason Huynh
I think Kirk's topic and any solution related to stats (int to long) should be resolved before cutting the branch? On Wed, Feb 13, 2019 at 7:48 PM Alexander Murmann wrote: > If there are no other takers, I can act as release manager for 1.9 and will > cut a release branch this week. > > > On Tue

Re: Geode 1.9 Release Manager

2019-02-14 Thread Jason Huynh
-6391, which is to fix a NPE while propagating región > > destroy and invalidate region messages. > > > > Regards > > Naba > > > > > > On Thu, Feb 14, 2019 at 9:06 AM Jason Huynh wrote: > > > >> I think Kirk's topic and any solut

Re: Release branch for Apache Geode 1.9.0 has been cut

2019-02-19 Thread Jason Huynh
Correction, GEODE-6359 and GEODE-6404. On Tue, Feb 19, 2019 at 4:49 PM Jason Huynh wrote: > I still haven't gotten GEODE-6404 in... I assumed that the tickets from > the last thread were going to make it into this release? > > Also I think GEODE-6539 should be fixed, it looks

Re: Release branch for Apache Geode 1.9.0 has been cut

2019-02-19 Thread Jason Huynh
I still haven't gotten GEODE-6404 in... I assumed that the tickets from the last thread were going to make it into this release? Also I think GEODE-6539 should be fixed, it looks like an NPE that occurs when we process leave requests. On Tue, Feb 19, 2019 at 2:25 PM Sai Boorlagadda wrote: > M

Re: Release branch for Apache Geode 1.9.0 has been cut

2019-02-20 Thread Jason Huynh
roduced by new work being merged to develop. > > Do you mean GEODE-6369? > > On Tue, Feb 19, 2019 at 4:50 PM Jason Huynh wrote: > > > Correction, GEODE-6359 and GEODE-6404. > > > > On Tue, Feb 19, 2019 at 4:49 PM Jason Huynh wrote: > > > > > I still h

Re: Release branch for Apache Geode 1.9.0 has been cut

2019-02-27 Thread Jason Huynh
it? > > On Wed, Feb 20, 2019 at 9:08 AM Jason Huynh wrote: > > > Oh ok I thought I read that voting was going to start soon, so I thought > > I'd raise a concern about the tickets not being fixed yet. > > > > I meant this ticket https://issues.apache.org/jira/brow

Re: Release branch for Apache Geode 1.9.0 has been cut

2019-02-27 Thread Jason Huynh
gt; > On Wed, Feb 27, 2019 at 8:54 AM Jason Huynh wrote: > > > Hi Sai, > > > > Fix for GEODE-6404 is now on develop > > (2be6375a775b6b0d00d0c41a1e2a3bf4b8745a46) > > Would you be able to pull it into the 1.9 branch or would you like me > to? > > >

Re: Release branch for Apache Geode 1.9.0 has been cut

2019-02-27 Thread Jason Huynh
Ok GEODE-6404 is merged into release/1.9.0 branch commit 40b3d9c690257d3e0e9bccfa1677c8adcbdce096 (*HEAD -> **release/1.9.0*, *origin/release/1.9.0*) Author: Jason Huynh Date: Mon Feb 25 18:31:21 2019 + GEODE-6404: work around possible sync issue with computeIfAbsent (#3

Re: 1.9 release date

2019-03-01 Thread Jason Huynh
To Alexander's point, I'm use the latest geode snapshot and am seeing an issue that looks similar to (if not the same as) GEODE-3780 (but this one is closed). I'd like to explore this a bit more and decide if that should be reopened but I am not sure if it's not an issue important enough to wait fo

Re: defunct branches

2019-04-17 Thread Jason Huynh
Hi Bruce, I am unable to see the same branches on geode repo. I do see these branches on my personal fork but that's because I haven't updated my own personal fork in some time... Is there a chance that your origin is pointing to your personal fork and not the Apache Geode Repo? I am also unabl

Re: GEODE-6630 fix for release 1.9.0

2019-04-18 Thread Jason Huynh
+1 On Thu, Apr 18, 2019 at 1:48 PM Dan Smith wrote: > +1 - this looks like a good fix to get in if it was introduced in 1.9.0. > > -Dan > > On Thu, Apr 18, 2019 at 1:28 PM Eric Shu wrote: > > > I'd like to include the fix for the NPE. > > It is new in 1.9. > > > > Regards, > > Eric > > >

Re: [PROPOSAL]: Improve OQL Method Invocation Security

2019-06-24 Thread Jason Huynh
+1 I have some concerns about all of the different ways we configure geode to be secure, but that's a different issue ;-) Overall, very thorough proposal Juan! On Mon, Jun 24, 2019 at 4:22 PM Dan Smith wrote: > +1 > > This proposal looks good to me! > > On Mon, Jun 24, 2019 at 4:15 PM Udo Koh

Re: [PROPOSAL]: Improve OQL Method Invocation Security

2019-07-02 Thread Jason Huynh
Are security manager policies modifiable on the fly? Just wondering if someone decides they want to disallow or allow something, will they need to restart their vms/geode node? I think Dan pointed this out earlier in the thread, but just wanted to have us consider the original cve that led to the

Re: SSL Alias Support for JMX Connections

2019-08-09 Thread Jason Huynh
+1 On Thu, Aug 8, 2019 at 7:12 PM Owen Nichols wrote: > Hi Juan and Sai, thank you for bringing your concern. > > Geode's release process dictates a time-based schedule < > https://cwiki.apache.org/confluence/display/GEODE/Release+Schedule> to > cut release branches. The release/1.10.0 < > http

Re: [DISCUSS] Tweak to branch protection rules

2019-10-30 Thread Jason Huynh
+1, thanks Dan! On Wed, Oct 30, 2019 at 10:07 AM Aaron Lindsey wrote: > +1 > > - Aaron > > > On Wed, Oct 30, 2019 at 8:02 AM Ju@N wrote: > > > Perfect Naba, thanks for answering this. > > My vote is +1 then. > > > > On Wed, Oct 30, 2019 at 2:37 PM Nabarun Nag wrote: > > > > > The check box Dan

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

2019-10-30 Thread Jason Huynh
If we are going to allow overrides, then maybe what Owen is describing should occur. Make a request on the dev list and explain the reasoning. I don't think this has been done and a few have already been overridden. Also who has the capability to override and knows how. How is that determined?

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

2019-10-31 Thread Jason Huynh
Greetings, We have a pull request (https://github.com/apache/geode/pull/4250) that is running into a problem with stressNewTest. Mostly the tests that are being run are RollingUpgrade tests that take quite a bit of time to run the full suite. Because these tests are added/modified, the stressNew

Re: IncrementalBackupDistributedTest.testMissingMemberInBaseline hangs

2019-11-06 Thread Jason Huynh
I'm working on a fix and have a PR up for another hang in the same test that I think fixes this issue. https://github.com/apache/geode/pull/4255 On Wed, Nov 6, 2019 at 10:47 AM Kirk Lund wrote: > IncrementalBackupDistributedTest.testMissingMemberInBaseline is hanging > intermittently in the Dis

Re: Lucene upgrade

2019-11-06 Thread Jason Huynh
Hi Mario, I think there are a few ways to accomplish what Dan was suggesting...Dan or other's, please chime in with more options/solutions. 1.) We add some product code/lucene listener to detect whether we have old versions of geode and if so, do not write to lucene on the newly updated node unti

Re: Lucene upgrade

2019-11-06 Thread Jason Huynh
ch in indexing for some period of time have an > > impact? > > > > Not an ideal solution but it’s something. > > > > In my previous life we just deleted the indexes and rebuilt them on > > upgrade but that was specific to our application. > > > > -Jak

Re: Lucene upgrade

2019-11-06 Thread Jason Huynh
, completely. In either case, I'll support whoever implements the changes :-P On Wed, Nov 6, 2019 at 2:53 PM Jacob Barrett wrote: > > > > On Nov 6, 2019, at 2:16 PM, Jason Huynh wrote: > > > > Jake, -from my understanding, the implementation details of geode-lucene

Re: Lucene upgrade

2019-11-07 Thread Jason Huynh
ed. But usually customer will hold > on, silence or reduce their business throughput when > doing rolling upgrade. I wonder if it's a reasonable assumption. > > Overall, after compared all the 3 options, I still think option-2 is the > best bet. > > Regards > Ge

Re: [DISCUSS] - Upgrading from Spring 4 to Spring 5

2019-11-07 Thread Jason Huynh
+1 On Thu, Nov 7, 2019 at 1:28 PM Dan Smith wrote: > +1 > > On Thu, Nov 7, 2019 at 12:49 PM Jens Deppe wrote: > > > +1 > > > > On Wed, Oct 30, 2019 at 1:39 PM Udo Kohlmeyer wrote: > > > > > Sorry, > > > > > > To clarify... When we change the Spring version we would be looking at > > > looking

Re: Propose adding GEODE-7400 fix to 1.11 release

2019-11-11 Thread Jason Huynh
+1 On Mon, Nov 11, 2019 at 9:41 AM Kirk Lund wrote: > I propose merging the fix for GEODE-7400 (merged to develop today) to the > 1.11 release branch. > > My fix for GEODE-7330 (merged to develop in late October) introduced > GEODE-7400 which is the potential for RejectedExecutionException to be

Re: Quick turnaround needed, feedback on this DRAFT Nov board report

2019-11-13 Thread Jason Huynh
+1 On Wed, Nov 13, 2019 at 10:06 AM Udo Kohlmeyer wrote: > +1 > > Haven't double checked the numbers, but the rest LGTM > > On 11/13/19 9:49 AM, Karen Miller wrote: > > Draft board report for November 2019. Submitting in 2 hours! Quick > > feedback, please! > > > > ## Description: > > The miss

Re: Odg: gateway sender queue

2019-11-14 Thread Jason Huynh
+1 to Dan's suggestion On Thu, Nov 14, 2019 at 9:52 AM Dan Smith wrote: > I'm ok with adding a --cleanQueue option. > > However, I don't think it should default to be true, since that would be > changing the behavior of the existing command. It should default to false. > > -Dan > > On Thu, Nov 1

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

2019-11-22 Thread Jason Huynh
@Udo - I think Naba was asking why the original commit that broke the pipeline wasn't detected. I think instead of a vote email, an email alerting the dev list that an override needs to take place is still good to have. If nothing else, to identify areas that we might be able to improve with addi

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

2019-11-22 Thread Jason Huynh
+1 I think we are now at +114 thanks to jinmei's 100 ;-) On Fri, Nov 22, 2019 at 1:50 PM Mark Bretl wrote: > +1 > > On Fri, Nov 22, 2019 at 12:55 PM Nabarun Nag wrote: > > > +1 > > > > On Fri, Nov 22, 2019 at 12:51 PM Charlie Black > wrote: > > > > > this proposal == awesome sauce > > > > > >

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

2019-11-22 Thread Jason Huynh
on't, and the rest of the 90+ committers who don't care. > > Are you suggesting that every override now becomes a vote on the dev > list? Given that we don't have a real stance on whether we allow it or > not, maybe that is best UNTIL we hit a scenario where we cannot ge

Re: [DISCUSS/VOTE] Proposal to bring GEODE-7465 to release/1.11.0

2019-11-26 Thread Jason Huynh
+1 On Tue, Nov 26, 2019 at 11:34 AM Anilkumar Gingade wrote: > +1 > > On Tue, Nov 26, 2019 at 11:32 AM Udo Kohlmeyer wrote: > > > This is no-brainer > > > > *+1* > > > > On 11/26/19 11:27 AM, Owen Nichols wrote: > > > I would like to propose bringing “GEODE-7465: Set eventProcessor to > null >

Re: IndexType deprecation question

2019-12-02 Thread Jason Huynh
Hi Joris, How are you creating the index? If using the QueryService java api, there should be createKeyIndex() and createIndex() methods. These methods should create the primary key index and the functional index. I am not sure if there is an alternative in gfsh... it might still be using the I

Re: Odg: Lucene upgrade

2019-12-02 Thread Jason Huynh
t; BR, > Mario > > > > Šalje: Xiaojian Zhou > Poslano: 7. studenog 2019. 18:27 > Prima: geode > Predmet: Re: Lucene upgrade > > Oh, I misunderstood option-1 and option-2. What I vote is Jason's option-1. > > On Thu, Nov 7, 2019

Re: IndexType deprecation question

2019-12-02 Thread Jason Huynh
he deprecation should have been limited to the HASH > type on the IndexType ENUM instead of deprecating the complete ENUM. > > Thanks, Joris. > > On Mon, Dec 2, 2019 at 12:05 PM Jason Huynh wrote: > > > Hi Joris, > > > > How are you creating the index? If using th

Re: Odg: Odg: Lucene upgrade

2019-12-06 Thread Jason Huynh
2> tid=0x46] During normal > processing, unsuccessfully dispatched 1 events (batch #0)* > For VM0(server1) and VM2(server3) it has 14 entries, one is unsuccessfully > dispatched. > > I don't know why some events are successfully dispatched, some not. > Do you have any idea?

Request GEODE-7510/GEODE-7538 be cherry-picked into release 1.11

2019-12-11 Thread Jason Huynh
Hello, GEODE-7538 was highlighted as blocking the 1.11 release. This has now been addressed and propose that this gets merged over to release 1.11. This issue solves a few things, most notably: GEODE-7510 shows inconsistency between secondaries and primaries. GEODE-7538 showed operations not co

Re: Request GEODE-7510/GEODE-7538 be cherry-picked into release 1.11

2019-12-11 Thread Jason Huynh
he SHA of the commit? > > Thanks, > Mark > > > On Dec 11, 2019, at 11:02 AM, Jason Huynh wrote: > > > > Hello, > > > > GEODE-7538 was highlighted as blocking the 1.11 release. This has now > been > > addressed and propose that this gets merged ov

Re: Odg: Odg: Odg: Lucene upgrade

2019-12-11 Thread Jason Huynh
ated. I also tried to put just few > objects and it passed as it had enough time to create indexes. > Do we need to change this part to wait for repo, or put a lower number of > entries in tests? > > BR, > Mario > > > > -- > *Šalje:* Jason

Re: Odg: Odg: Odg: Odg: Lucene upgrade

2019-12-13 Thread Jason Huynh
In tests it tried to execute a query after all members are rolled. > > BR, > Mario > > -- > *Šalje:* Jason Huynh > *Poslano:* 11. prosinca 2019. 23:08 > *Prima:* Mario Kevo > *Kopija:* geode > *Predmet:* Re: Odg: Odg: Odg: Lucene upgrade

Re: Reviewer for GEODE-7534: Add example for query with bind params (documentation)

2019-12-17 Thread Jason Huynh
Hi Alberto, Looks like Dan and Dave have both reviewed it. I took a look and didn't see anything wrong so I squash merged it in. Thanks! -Jason On Fri, Dec 13, 2019 at 3:30 AM Alberto Gomez wrote: > Hi, > > I'd appreciate some extra reviewer (I already had one, thanks @Dave > Barnes) and if e

Re: Proposal to including GEODE-7593 in release/1.11.0

2019-12-19 Thread Jason Huynh
+1 On Thu, Dec 19, 2019 at 10:05 AM Owen Nichols wrote: > GEODE-7593 fixes a memory leak where indexes could retain references to > pdx values when eviction should have released that memory. > > This is not a new issue, but is critical because system stability is > threatened when eviction does

Re: [DISCUSS] abandon branch protection rules

2019-12-27 Thread Jason Huynh
I feel the frustration at times, but I do also think the ci/pipelines are improving, breaking less often. I'm ok with the way things are for the moment On Fri, Dec 27, 2019 at 1:47 PM Owen Nichols wrote: > In October we agreed to require at least 1 reviewer and 4 passing PR > checks before a PR

Re: [DISCUSS] abandon branch protection rules

2019-12-27 Thread Jason Huynh
cific tests in the future. On Fri, Dec 27, 2019 at 2:58 PM Jason Huynh wrote: > I feel the frustration at times, but I do also think the ci/pipelines are > improving, breaking less often. I'm ok with the way things are for the > moment > > On Fri, Dec 27, 2019 at 1:47 PM Owe

Re: RFC - Logging to Standard Out

2020-01-08 Thread Jason Huynh
+1 On Wed, Jan 8, 2020 at 1:21 PM Dan Smith wrote: > +1. Looks good! > > -Dan > > On Wed, Jan 8, 2020 at 12:56 PM Blake Bender wrote: > > > +1 - this is also a todo item for the native client, I think. NC has a > bug > > in logging which is in my top 3 for "most irritating," as well, which is

Re: Odg: ParallelGatewaySenderQueue implementation

2020-01-27 Thread Jason Huynh
Some additional info/context from the PR that is blocked by this issue: Although we have GFSH stop, it can still be used on an individual node. We just publish a caution, but it looks like we still allowed it due to having some users using it: CAUTION: Use caution with the stop gateway-sender co

Re: OQL Method Authorizer Blog

2020-02-14 Thread Jason Huynh
Great job Juan! Very informative and detailed read. On Fri, Feb 14, 2020 at 4:43 AM Nabarun Nag wrote: > Hi Geode Community, > > Please do visit the blog that Juan Ramos has put up on the OQL Method > Authorizer : > > https://jujoramos.blogspot.com/2020/02/pluggable-oql-method-authorization.htm

Re: Question about Hash Indexes

2020-02-19 Thread Jason Huynh
Hi Mario, >From my understanding: 1.) The Hash Index was implemented unlike a traditional hash index, instead it is more of a memory saving index that uses hashing to not store keys. When the backing data structure needs to expand, it needs to rehash a lot of data and this can be detrimental at ru

[PROPOSAL] Include fix for GEODE-7763 into release 1.12.0

2020-03-18 Thread Jason Huynh
Hello Dev list, I'd like to include a fix for GEODE-7763 in release 1.12.0. The change removes the call to exportValue, preventing a serialization, when no clients are waiting for the specific event. The reason why I think it should be in the release is that we noticed a negative effect on perform

Re: [DISCUSS] Client side configuration for a SNI proxy

2020-03-19 Thread Jason Huynh
+1 On Thu, Mar 19, 2020 at 7:27 AM Joris Melchior wrote: > +1 > > On Mon, Mar 16, 2020 at 6:33 PM Dan Smith wrote: > > > Hi all, > > > > A new RFC is up for this feature > > > > > https://cwiki.apache.org/confluence/display/GEODE/Client+side+configuration+for+a+SNI+proxy > > . > > > > > > Pleas

Re: [ANNOUNCE] Github Wiki and Issus are now activated on geode-kafka-connector

2020-03-24 Thread Jason Huynh
Awesome, thanks for doing all the leg work! On Tue, Mar 24, 2020 at 1:42 PM Nabarun Nag wrote: > Hi, > > Issues are wiki pages are now active on the geode-kafka-connector > repository. Thank you all for the kind votes. > > Regards > Nabarun Nag >

Re: RFC - Gateway sender to deliver transaction events atomically to receivers

2020-03-25 Thread Jason Huynh
I put some comments on the proposal on the wiki. btw what are we voting on? Just curious as I wasn't sure if we were voting for the current proposal or whether we should continue this discussion? I like the idea of having transactional ops be sent together in a batch if possible and it would be

Re: [Discuss] Cache.close synchronous is not synchronous, but code still expects it to be....

2020-04-14 Thread Jason Huynh
The isClosed flag and method might be used currently more as an isClosing. The GemFireCacheImpl.isClosed() method is actually returning isClosing. Whatever change to isClosed that will be made, will have to properly handle cases where it's meant to be treated as isClosing(). On Tue, Apr 14, 2020 a

[PROPOSAL] Backport usability improvements to support 1.13 branch

2020-09-23 Thread Jason Huynh
Hello, I’d like to merge the pull request: https://github.com/apache/geode/pull/5524 into a support 1.13 branch. The commits are focused on a few usability improvements for Geode that were thought to have made it into 1.13 but actually did not make it. What this pull request back ports: *

Re: [VOTE] Apache Geode 1.13.1.RC2

2020-11-17 Thread Jason Huynh
+1 Ran gfsh, created cluster, create region On 11/16/20, 10:29 AM, "Dan Smith" wrote: +1 Looks good to me! I ran the geode-release-check against it, looked for binary artifacts, checked the pipeline. -Dan From: Dick Cavender Sent: Thu

Re: Kafka Summit:- Geode Kafka Connector

2021-01-07 Thread Jason Huynh
Hi Ashish, Are you asking for someone to present, or are you asking if it is ok for you to present? I don't think there are any restrictions on presenting, hopefully someone will correct me if I am wrong. I think the call for papers is still open but closing relatively soon? So if you plan

Re: Inputs for efficient querying

2021-02-10 Thread Jason Huynh
Hi Ankit, I haven't had time to try this out but hopefully the answers get you on the correct path... > 1. How can i form an OQL query (syntax) to fetch the latest row based on > MAX(versionId). 1.) maybe a nested query or use order by? “Select x,y,z from /data-region d where d.versionId

Re: Question about Map indexes

2021-02-11 Thread Jason Huynh
Hi Alberto, I haven't checked the PR yet, just read through the email. The first thought that comes to mind is when someone does a != query. The index still has to supply the correct answer to the query (all entries with null or undefined values possibly) I'll try to think of other cases whe

Re: Last call on 1.2.0 (and fixing test failures)

2017-05-15 Thread Jason Huynh
GEODE-2900 would be nice to get in for Lucene integration. I hope to get it checked in today On Mon, May 15, 2017 at 10:24 AM Swapnil Bawaskar wrote: > I think we should also wait for GEODE-2836 > > On Mon, May 15, 2017 at 8:58 AM Karen Miller wrote: > > > Let's finish GEODE-2913, the document

Re: Last call on 1.2.0 (and fixing test failures)

2017-05-15 Thread Jason Huynh
> > > > > > > On Mon, May 15, 2017 at 11:26 AM, Jason Huynh wrote: > > > >> GEODE-2900 would be nice to get in for Lucene integration. I hope to > get > >> it checked in today > >> > >> On Mon, May 15, 2017 at 10:24 AM Swapnil Bawaskar

Re: Broken: apache/geode#2632 (develop - 7da9047)

2017-05-22 Thread Jason Huynh
t; > Duration: 9 minutes and 51 seconds > Commit: 7da9047 (develop) > Author: Jason Huynh > Message: GEODE-269: Removing deprecated API's from FunctionService. > > This closes #511 > > View the changeset: > https://github.com/apache/geode/compare/4e3a4d762d7c...7

Re: Build failed in Jenkins: Geode-nightly #864

2017-06-12 Thread Jason Huynh
I've created a ticket for the failing wan test: GEODE-3066 On Mon, Jun 12, 2017 at 8:34 AM Apache Jenkins Server < jenk...@builds.apache.org> wrote: > See > >

Re: continuous query internal mechanism questions

2017-08-15 Thread Jason Huynh
I am not quite sure how native client registers cqs. From my understanding: with the java api, I believe there is only one message (ExecuteCQ message) that is executed on the server side and then replicated to the other nodes through the profile (OperationMessage). It seems the extra ExecuteCQ mes

Re: Indxes and hints

2017-08-28 Thread Jason Huynh
ollector)); > reply.setTimeout(timeout); > > GfErrType err = GF_NOERR; > err = m_tccdm->sendSyncRequest(msg, reply); .. > > And sendSyncRequest: > ... > > for (std::vector::iterator ep = m_endpoints.begin(); ep != > m_endpoints.end(); ++ep) { > if ((*ep

Re: Query mechanism

2017-08-28 Thread Jason Huynh
; err = m_tccdm->sendSyncRequest(msg, reply); .. > > And sendSyncRequest: > ... > > for (std::vector::iterator ep = m_endpoints.begin(); ep != > m_endpoints.end(); ++ep) { > if ((*ep)->connected()) { > (*ep)->setDM(this); > opErr = sendRequestToEP(request,

Re: Indxes and hints

2017-08-30 Thread Jason Huynh
r > Sent: Tuesday, August 29, 2017 4:02 PM > To: dev@geode.apache.org > Subject: RE: Indxes and hints > > Thank you Jason :-) > > -Original Message- > From: Jason Huynh [mailto:jhu...@pivotal.io] > Sent: Monday, August 28, 2017 7:24 PM > To: dev@geode.apache.org

Re: Indxes and hints

2017-09-05 Thread Jason Huynh
quot;? > > I also noticed that even if I query on A and B, and ORDER BY B - it seemed > to perform the query on B only, at least in a separate stage. Why is that? > > > > > -Roi > > -Original Message- > From: Jason Huynh [mailto:jhu...@pivotal.io] > Sent:

Re: [DISCUSS] Addition of isValid API to Index interface

2017-09-10 Thread Jason Huynh
1.) Does anyone know of a way to do a rollback where the put is already reflected in the region? If that is the desired behavior, then perhaps we will have to live with the current (leaving the region and indexes in a bad state, wan and other callbacks that occur after index maintenance will not

Re: [DISCUSS] Addition of isValid API to Index interface

2017-09-11 Thread Jason Huynh
Hi Mike, I think the concern was less about the security portion but rather if any exception occurs during index update, right now, the region gets updated and the rest of the system (index/wan/callbacks) may or may not be updated. I think Naba just tried to provide an example where this might occ

Re: [DISCUSS] Addition of isValid API to Index interface

2017-09-11 Thread Jason Huynh
el Stolz wrote: > > > > > > Great, that's exactly the behavior I would expect. > > > > > > Thanks. > > > > > > -- > > > Mike Stolz > > > Principal Engineer, GemFire Product Manager > > > Mobile: +1-631-835-4771 <(631

Re: [DISCUSS] Clean build takes 10minutes to complete now

2017-09-11 Thread Jason Huynh
I was speaking with Jens and he is working on getting the full product install up to the apache maven repo. We could then use this instead of downloading the zip files manually. This would allow gradle to cache the full product install in the local repo (similar to the other jars that are being p

Re: [DISCUSS] Clean build takes 10minutes to complete now

2017-09-15 Thread Jason Huynh
For the original issue, where the old version is pulling down the old versions during compilation time, we have a pull request to use the maven repo: https://github.com/apache/geode/pull/790 The rest of the tests added for the session state are marked as @Category ({DistributedTest.class, Backward

Re: [DISCUSS] Framework for concurrency tests

2017-09-15 Thread Jason Huynh
+1 to Dan's Changes but also +1 to Galen's suggestion. JPF looks like it might take a bit to run all the different states even for a small interleaving of code (maybe we can tune/configure it though). Or we can mark these as a different category and not run as a "UnitTest" On Fri, Sep 15, 2017 a

Re: [DISCUSS] Addition of isValid API to Index interface

2017-09-21 Thread Jason Huynh
t; > >> index updates in a transactional way, silently invalidating > > indexes > > > > may > > > > > >> not > > > > > >> be acceptable... > > > > > >> > > > > > >> -An

[DISCUSS] Removal of "Submit an Issue" from Geode webpage

2017-09-29 Thread Jason Huynh
I'd like to remove the "Submit an Issue" button/script attached to the site. We occasionally get JIRA tickets that come through the Apache Geode website through the "Submit an Issue" Button However these tickets are being created through a script, and this sets Gregory Chase as the reporter and a

Re: Rebase and squash before merging PRs

2017-10-05 Thread Jason Huynh
I think we can also use "squash and merge" if wanting to squash commits before merging. This would allow you not to have to force push every time. On Thu, Oct 5, 2017 at 3:15 PM Jinmei Liao wrote: > On the PR UI page, you can do that by pull down the the menu when you are > ready to merge. Reme

Permissions to edit the wiki

2017-10-27 Thread Jason Huynh
Hi, I would like to be able to edit the wiki for Geode and I don't think I have the correct permissions at this time. Would someone be able to give me the permissions to do so? My username in confluence is huynhja. Thanks, -Jason

Re: Permissions to edit the wiki

2017-10-27 Thread Jason Huynh
Thanks! On Fri, Oct 27, 2017 at 3:08 PM Dan Smith wrote: > You should have permissions now. > > -Dan > > On Fri, Oct 27, 2017 at 1:26 PM, Jason Huynh > wrote: > > > Hi, > > > > I would like to be able to edit the wiki for Geode and I don't think

Re: [Discussion] SQL+Streaming/JDBC as one of the unified interfaces

2017-11-13 Thread Jason Huynh
Hi Christian, I don't know much about Calcite and haven't had a chance to try out your adapter yet but it sounds like a neat idea. Will your talk be recorded and available after the Summit? Also for question 1. Would you be interested to have the adapter as part of Geode's code ecosystem? Do you

[DISCUSS] changes to registerInterest API

2017-11-16 Thread Jason Huynh
For GEODE-3813 : Region registerInterest API usage of type parameters is broken The current API to registerInterest allows a special string token “ALL_KEYS” to be passed in as the parameter to reg

Re: [DISCUSS] changes to registerInterest API

2017-11-17 Thread Jason Huynh
gt; > > > > > > > -- > > > Mike Stolz > > > Principal Engineer, GemFire Product Lead > > > Mobile: +1-631-835-4771 <(631)%20835-4771> <(631)%20835-4771> > > > > > > On Thu, Nov 16, 2017 at 7:02 PM, Anilkumar Gingade < >

Re: [DISCUSS] changes to registerInterest API

2017-11-17 Thread Jason Huynh
List is no different. Java's type erasure essentially equates var > args > > too "Object..." (or Object[]) and the List to List (or a List of > > Objects, > > essentially like if you just did this... List) So, while the > > compiler ensures compile-time typ

Re: [DISCUSS] changes to registerInterest API

2017-11-17 Thread Jason Huynh
egisterInterest(keys); *// calls the > Region.registerInterest(:Iterable) method.* > > Alternatively, this would also be allowed... > > *Set* keys = ... > > region.registerInterest(keys); > > > On Fri, Nov 17, 2017 at 11:44 AM, Jason Huynh wrote: > > &g

[DISCUSS] FunctionAdapter incompatible serialVersionUID

2017-11-27 Thread Jason Huynh
This is a discussion for the fix to GEODE-4008: InvalidClassException when deserializing FunctionAdapter from pre Geode clients There was a change to deprecate FunctionAdapter in Geode (before 1.0), and this also removed the method signatures in the class. This caused Java to generate a new serial

Re: [DISCUSS] FunctionAdapter incompatible serialVersionUID

2017-11-28 Thread Jason Huynh
> > > > > > > On Nov 28, 2017, at 11:40 AM, Patrick Rhomberg < > prhomb...@pivotal.io > > > > > > > wrote: > > > > > > > > > > +1 to removing a long-deprecated class from the Geode side. > > > > >

Re: [DISCUSS] FunctionAdapter incompatible serialVersionUID

2017-11-28 Thread Jason Huynh
functionAdapter. This should only affect users that extend FunctionAdapter and execute functions by serializing them to the server from the client. If they execute by id it should not run into this problem... On Tue, Nov 28, 2017 at 5:20 PM Jason Huynh wrote: > Dan, yeah, the suggested change in

Re: [DISCUSS] changes to registerInterest API

2017-11-30 Thread Jason Huynh
_KEYS, what about an API that doesn't require a token or all keys: > > public void registerInterestForAllKeys(); > > On Fri, Nov 17, 2017 at 1:24 PM, Jason Huynh wrote: > > > Thanks John for the clarification! > > > > On Fri, Nov 17, 2017 at 1:12 PM John Blum wro

Re: [DISCUSS] changes to registerInterest API

2017-11-30 Thread Jason Huynh
mposite" Key, but then define a > > CompositeKey class type with properly implemented equals(:Object) and > > hashCode():int methods. > > > > For the most part, Keys should really only ever be simple Scalar values > > (e.g. Long, String, etc). > > > > -j > &g

Apache geode nightly failures

2017-12-01 Thread Jason Huynh
Has anyone been getting emails for the nightly builds and failures (My last build email was 1027 and I think the latest is 1030)? It looks like the last few have been a mess with Out of Memory exceptions and I think they were all ran on H34. Should that machine be black listed? If so, would some

Re: [DISCUSS] FunctionAdapter incompatible serialVersionUID

2017-12-05 Thread Jason Huynh
7 at 7:03 PM, Jacob Barrett > > wrote: > > > > > Since the class was deprecated and is technically only there for > pre-1.0 > > > compatibly then the behavior of this class should be consistent with > the > > > pre-1.0 version. This will break 1.0 to 1

[DISCUSS] Proposal to Deprecate Hash Index

2017-12-05 Thread Jason Huynh
This is a proposal to deprecate existing Hash Index and deprecate the create hash index apis. Currently the Hash Index name causes confusion. It is not a traditional hash look up index, but more of memory savings index. The index does not store index keys in memory and must hash the keys every t

Re: Next release: 1.4.0

2018-01-04 Thread Jason Huynh
Hi Swapnil, GEODE-4140 was just marked for 1.4. I think part of GEODE-4140 should be fixed because the process.ClusterConfigurationNotAvailableException should probably be reinstated. If others don't think it's needed then feel free to remove the fix tag. -Jason On Thu, Jan 4, 2018 at 4:38 PM

Re: Next release: 1.4.0

2018-01-05 Thread Jason Huynh
s jobs to run the `build` and > > `updateArchives` tasks since those still have value. > > > > Anthony > > > > > > > On Jan 4, 2018, at 5:03 PM, Alexander Murmann > > wrote: > > > > > > The Concourse pipeline seems much more reliable a

[PROPOSAL] Add mod and arithmetic functionality to OQL

2018-02-06 Thread Jason Huynh
This is a proposal to add the ability to execute arithmetic operations on numeric fields in an OQL query. The corresponding jira ticket is: https://issues.apache.org/jira/browse/GEODE-4327 The operation symbols that will be added are mod, %, +, -, /, *. These will correspond to modulo, modulo, a

Re: [PROPOSAL] Add mod and arithmetic functionality to OQL

2018-02-06 Thread Jason Huynh
Based on the oql.g file I think it would but I can add some tests to be sure On Tue, Feb 6, 2018 at 2:04 PM Anthony Baker wrote: > Will the OQL engine obey operator precedence rules? > > Anthony > > > > On Feb 6, 2018, at 10:22 AM, Jason Huynh wrote: > > > &

Re: Geode unit tests 'develop/DistributedTest' took too long to execute

2018-02-12 Thread Jason Huynh
I logged a ticket for this hang: https://issues.apache.org/jira/browse/GEODE-4650 From what I can tell, it looks like some sort of race condition where the DLockService is stuck trying to clearGrantor. On Fri, Feb 9, 2018 at 10:20 PM wrote: > Pipeline results can be found at: > > Concourse: > h

Re: [DISCUSS] changes to registerInterest API

2018-02-20 Thread Jason Huynh
rs created a single, CompositeKey > >> class type, with properly implements equals and hashCode methods, as I > >> pointed out. > >> > >> I don't see any advantage in using a java.util.Collection as a key over > >> implementing a CompositeKey t

Re: [Proposal] Thread monitoring mechanism

2018-02-21 Thread Jason Huynh
I am assuming this would be for all thread/thread pools and not specific to Function threads. I wonder what the impact would be for put/get operations or are we going to target specific operations. On Tue, Feb 20, 2018 at 1:04 AM Gregory Vortman wrote: > Hello team, > One of the most severe i

Re: [PROPOSAL] use default value for validate-serializable-objects in dunit

2018-03-15 Thread Jason Huynh
+1 agreement with Kirk and Sean. Any non default configuration should probably have it's own set of tests. I can understand some exploratory work where someone might want to run the whole precheckin with a non default value to help identify areas that they may have missed or are unexpectedly affec

Re: Index on Region

2018-04-16 Thread Jason Huynh
Hi Jinmei, I am not sure whether these elements were deprecated or not. I know that they were at one time valid and a user could specify the following in their app at one point: I believe the "new" way to do this would be: How would deprecation for this work? Would your roll a new vers

Re: [Proposal]: behavior change when region doesn't exist in cluster configuration

2018-04-27 Thread Jason Huynh
Hi Jinmei and Naba, I don't think you can define two regions with the same name and different types. We would throw an IllegalStateException for the node that tried to create the region second. At least that was the behavior I was seeing when I tried to create a replicate region and a partitione

Re: [Proposal]: behavior change when region doesn't exist in cluster configuration

2018-04-27 Thread Jason Huynh
*correction to my last email, I was using java api and not cache.xml On Fri, Apr 27, 2018 at 3:40 PM Jason Huynh wrote: > Hi Jinmei and Naba, > > I don't think you can define two regions with the same name and different > types. We would throw an IllegalStateException for th

  1   2   3   4   >