Re: Rebalancing on jvm recovery

2018-01-22 Thread Dan Smith
Hi Igor, For your first question, redundancy recovery by default starts happening ansynchronously after your member starts up. So I would expect if you wait long enough you will see the full redundancy is restored. You can control this behavior with the startup-recovery-delay setting. For your se

Re: [DISCUSS] Benchmarks module package structure

2018-01-08 Thread Dan Smith
I think this module was specifically added for running JMH benchmarks since it's pulling in the JMH plugin. JMH is framework for easily writing microbenchmarks. I think it makes sense to put JMH benchmarks in the same package as the code under tests, since you may end up writing a microbenchmark f

Re: Next release: 1.4.0

2018-01-05 Thread Dan Smith
40 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. > > >

Re: Next release: 1.4.0

2018-01-04 Thread Dan Smith
Our process up to this point has been to not ship until the jenkins builds on the release branch pass. We've been experimenting with concourse in parallel with jenkins, but the jenkins builds on develop at least are still pretty messy. How are we going to ship this release? Should both be passing?

Re: Default branch for geode-examples repo

2018-01-02 Thread Dan Smith
We merge develop to master for geode-examples whenever we cut a release. When we release, we release all of the repos at once - currently just geode and geode-examples since geode-native isn't ready to release yet. The master branch of geode-examples builds and run against the latest geode release.

Re: Permission to edit tickets in Jira

2017-12-21 Thread Dan Smith
Hi Barbara, You should have access now. Thanks! -Dan On Thu, Dec 21, 2017 at 2:02 PM, Barbara Pruijn wrote: > Hi, > > Can someone please provide me with permission in Jira to edit tickets to > update components and Fix Version/s fields, etc? > > Thanks, > Barbara > Gemfire M&M PM

Re: Write access to wiki

2017-12-21 Thread Dan Smith
Done. You should have access now. -Dan On Thu, Dec 21, 2017 at 2:01 PM, Alexander Murmann wrote: > Hi everyone, > > Can I please get write access to the wiki? > > My username is "amurmann" > > Thank you! >

Re: Geode Jira/GitHub Integration configuration changes

2017-12-18 Thread Dan Smith
Hi Kirk, I think this is will probably require a request to infra. You can file a JIRA with infra assuming we have consensus to get rid of the diffs (huge +1 from me:) Also, please don't cross post with private@geode. The private list should *only* be used for things that can't be discussed publi

Re: Permissions to Add Content in the Wiki

2017-12-13 Thread Dan Smith
Hi Juan, You should have permissions now! Thanks, -Dan On Wed, Dec 13, 2017 at 3:33 AM, Juan José Ramos wrote: > Hello team, > > Can I get permissions to create content on the wiki?. My username is > *jujoramos*. > Best regards. > > -- > Juan José Ramos Cassella > Senior Technical Support Engi

Re: Debugging intermittent dunit failures

2017-12-11 Thread Dan Smith
n trying to reuse > distributed system across multiple dunit tests. The latter just isn't worth > the headache and trouble that it causes when static vars or constants or > disk artifacts pollute later tests. > > On Mon, Dec 11, 2017 at 1:42 PM, Dan Smith wrote: > > > One o

Re: Default branch for geode should be 'develop'

2017-12-11 Thread Dan Smith
+1 -Dan On Mon, Dec 11, 2017 at 1:40 PM, Jens Deppe wrote: > Currently, github has the default branch for apache/geode set to 'master'. > (which is why you need to point the relevant branch to 'develop' for every > new PR). > > Do we agree that this should be set to 'develop'? > > (I've message

Re: Debugging intermittent dunit failures

2017-12-11 Thread Dan Smith
One other thing you can do is look for the below line in the logs of your failure. These are the tests that ran in the same JVM as your tests. This won't help if your tests are getting messed up by disk artifacts or port issues, but if it is some JVM state left by a previous test it would be in thi

Re: [DISCUSS] Proposal to Deprecate Hash Index

2017-12-05 Thread Dan Smith
+1 -Dan On Tue, Dec 5, 2017 at 4:28 PM, Jason Huynh wrote: > 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

Re: DISCUSS: Deprecating and replacing current serializable string encodings

2017-12-04 Thread Dan Smith
at. All other clients should be able to deserialize that. -Dan On Fri, Dec 1, 2017 at 5:35 PM, Jacob Barrett wrote: > On Fri, Dec 1, 2017 at 4:59 PM Dan Smith wrote: > > > I think I'm kinda with Mike on this one. The existing string format does > > seem pretty gnarly.

Re: DISCUSS: Deprecating and replacing current serializable string encodings

2017-12-01 Thread Dan Smith
I think I'm kinda with Mike on this one. The existing string format does seem pretty gnarly. But the complexity of implementing and testing all of the backwards compatibility transcoding that would be required in order to move to the new proposed format seems to be way more work with much more poss

Re: [DISCUSS] changes to registerInterest API

2017-11-30 Thread Dan Smith
order > > > >> > > > > collection > > > >> > > > > > of > > > >> > > > > > > elements. > > > >> > > > > > > > > > >> > > > > > > This ("*It might e

Re: [DISCUSS] FunctionAdapter incompatible serialVersionUID

2017-11-28 Thread Dan Smith
I agree I don't think we can get rid of FunctionAdapter until the next major version. I was thinking FunctionAdapter is rather widely used, but then I'm surprised no one has hit this yet. All of the options kinda suck here - either pre 1.0 users have a compatibility issue or 1.0-1.3 users do. Wit

Re: Permissions to create content on wiki

2017-11-28 Thread Dan Smith
You should have access now. -Dan On Tue, Nov 28, 2017 at 9:45 AM, Fred Krone wrote: > Hi, > > Could I get permissions to create content on the wiki? My username is > fkrone. > > Thanks, > -Fred >

Re: [DISCUSS] changes to registerInterest API

2017-11-16 Thread Dan Smith
I don't really like the regex option - it implies that your keys are all strings. Will any other regular expressions work on non string objects? registerInterestAllKeys() seems like a better option. -Dan On Thu, Nov 16, 2017 at 4:34 PM, Michael Stolz wrote: > I don't like the vararg option. > I

Re: Need "edit" access to update Network Best Practices page

2017-11-10 Thread Dan Smith
otal.io/> | Mon-Fri 9:00am to > 5:30pm MT | 1-877-477-2269 <(877)%20477-2269> > > > On Nov 10, 2017, at 3:50 PM, Dan Smith wrote: > > Tod - what's your username on the wiki? If you don't have an account yet > you can create one. If you give us your usern

Re: Need "edit" access to update Network Best Practices page

2017-11-10 Thread Dan Smith
Tod - what's your username on the wiki? If you don't have an account yet you can create one. If you give us your username we can give you access. -Dan On Fri, Nov 10, 2017 at 2:41 PM, Bruce Schuchardt wrote: > Who can give Tod edit access to Confluence? He has some contributions > he'd like to

Re: Test Coverage Mapping

2017-11-10 Thread Dan Smith
Sorry, wrong link. I think they updated analysis.apache.org to this: https://builds.apache.org/analysis -Dan On Fri, Nov 10, 2017 at 2:00 PM, Dan Smith wrote: > We have some support in our gradle build to run tests coverage manually > with something like this: > > ../gradle

Re: Test Coverage Mapping

2017-11-10 Thread Dan Smith
We have some support in our gradle build to run tests coverage manually with something like this: ../gradlew clean test integrationTest distributedTest acceptanceTest -PcodeCoverage jacocoOverallTestReport This should generate a code coverage report. At one point I know we were talking about int

Re: [DRAFT] Geode Board Report November 2017

2017-11-10 Thread Dan Smith
Looks good to me! We did switch from git.apache.org to gitbox.apache.org and github this quarter. That may partly explain changes in the mailing list message counts if we get a getting fewer automated emails with the new system. -Dan On Thu, Nov 9, 2017 at 5:15 PM, Joey McAllister wrote: > Hi

Re: Permissions To View Builds?

2017-11-07 Thread Dan Smith
I think you just have the wrong url. Try this (job, not jobs): https://builds.apache.org/job/Geode-nightly/ On Tue, Nov 7, 2017 at 12:35 PM, Michael William Dodge wrote: > I tried to check on the nightly builds via https://builds.apache.org/ > jobs/Geode-nightly/

Re: [DISCUSS] CI improvements

2017-11-02 Thread Dan Smith
On Thu, Nov 2, 2017 at 11:58 AM, Sean Goller wrote: > Given the length of time precheckin seems to run, would it make sense to > break it up? > > -Sean. > Sure, as long as we don't miss anything :) -Dan

Re: [DISCUSS] CI improvements

2017-11-02 Thread Dan Smith
Looks good. Should we go ahead and change this to run precheckin instead of build? -Dan On Thu, Nov 2, 2017 at 9:53 AM, Anthony Baker wrote: > If you’d like to check this out, here’s the PR containing the pipeline and > job scripts: > https://github.com/apache/geode/pull/1006 > > And the pipeli

Re: Removing the old Authenticator/AccessControl interfaces

2017-10-30 Thread Dan Smith
Are you saying these interfaces were already deprecated in 1.0? If so, then I think we can remove them. If they have been deprecated after 1.0, I think we have to wait until 2.0 to remove them. -Dan On Mon, Oct 30, 2017 at 1:18 PM, Jinmei Liao wrote: > The old security framework, basically Auth

Re: Permissions to edit the wiki

2017-10-27 Thread Dan Smith
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 I have > the correct permissions at this time. Would someone be able to give me the > permissions to do so? My username in c

Re: [VOTE] Apache Geode release - 1.3.0 RC4

2017-10-26 Thread Dan Smith
+1 - passed geode-release-check https://github.com/upthewaterspout/geode-release-check On Thu, Oct 26, 2017 at 2:17 PM, Bruce Schuchardt wrote: > Thanks Anthony > > > > On 10/26/17 1:48 PM, Anthony Baker wrote: > >> From the release steps page: >> https://cwiki.apache.org/confluence/display/GE

Re: [VOTE] Apache Geode release - 1.3.0 RC3

2017-10-25 Thread Dan Smith
Looks like this RC is still a no go due to the missing signatures Anthony pointed out. Swap, when you fix that can you create a new release candidate or at least send out a new VOTE email? It's a bit confusing what the state of this VOTE is and what the deadline is. -Dan On Wed, Oct 25, 2017 at 1

Re: Geode - release 1.3 sourcing an artifact from AWS S3

2017-10-23 Thread Dan Smith
Hi Brian, Good catch! It's not something deliberately nefarious - I added java pathfinder as a dependency for writing concurrency tests, but unfortunately jpf itself doesn't have a good maven artifact to use. We could make a maven artifact with the jpf binary as part of the jpf plugin from plugin

Re: Native client examples

2017-10-23 Thread Dan Smith
po)... > > -Anil. > > > On Mon, Oct 23, 2017 at 9:18 AM, Jacob Barrett > wrote: > > > On Fri, Oct 20, 2017 at 6:02 PM Dan Smith wrote: > > > > > However I do think we should be compiling (and hopefully running!) > > > these examples so that they

Native client examples

2017-10-20 Thread Dan Smith
Hi, I noticed some C# examples showed up in the geode-examples project. Awesome! However I do think we should be compiling (and hopefully running!) these examples so that they don't rot over time. Especially since I believe we are still evolving the native API with breaking changes and have not a

Re: [Discuss] CliStrings

2017-10-20 Thread Dan Smith
+1 for removing it. I do think it would be nice if we add localization in the future. But I don't really like the idea of leaving stuff in our code just in case we decide to implement a feature in the future - we might not even want what we left in there! In this case even with localization we mig

Re: [VOTE] Apache Geode release - 1.3.0 RC3

2017-10-19 Thread Dan Smith
+1 Build passed, release check passed, examples are present, I don't see us shipping jars we shouldn't. Looks good to me! BTW, even though the actual build passed, the jenkins job failed with the below error about archiving results. Who has access to fix this? BUILD SUCCESSFUL > > Total time: 15

Re: [VOTE] Apache Geode release - 1.3.0 RC2

2017-10-17 Thread Dan Smith
-1 without the examples. We *could* theoretically be releasing the examples separately, but that's not our current release process. The release steps include releasing the examples - https://cwiki.apache.org/confluence/display/GEODE/Release+Steps. I don't think the examples release needs to wait

Re: Internal package name structure

2017-10-09 Thread Dan Smith
+1 for #2 We will need to be careful refactoring existing code if classes are sent over the wire. -Dan On Mon, Oct 9, 2017 at 10:36 AM, Udo Kohlmeyer wrote: > Hi there Geode devs, > > Whilst going through the code base I found that we have 2 differing > approaches of how we classify (or packag

Geode configuration (was: New client protocol configuration)

2017-10-02 Thread Dan Smith
Let's move this discussion about the best way to configure geode off of this thread about the client protocol. I do agree with John that we should move away from xml. Right now if I add a new configuration option I have to: - add it to the java API - Add it to the xsd - write code to do xml pa

Re: New client protocol configuration

2017-10-02 Thread Dan Smith
uration options are secondary and optional (as needed). > > Therefore, given an API-first approach, the other configuration formats and > options become more apparent (providing the API was designed with the right > abstractions in the first place). > > $.0.02, > -j > > > > On

Re: New client protocol configuration

2017-10-02 Thread Dan Smith
One thing to think about - if the new protocol doesn't support two-way authentication maybe we should throw an exception if the user sets ssl-require-authentication=true? We definitely don't want to lie to the user and pretend that we are providing some level of security which we are not. I'm assu

Re: New client protocol configuration

2017-10-02 Thread Dan Smith
Security configuration for this new protocol should should be done in a way that is consistent with existing SSL related properties. See https://geode.apache.org/docs/guide/12/managing/security/implementing_ssl.html. In this case, maybe this new protocol should be use the same configuration as the

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

2017-09-29 Thread Dan Smith
+1 - I think it would be better to just encourage users to send issues/questions to the users list initially. -Dan On Fri, Sep 29, 2017 at 11:08 AM, Michael William Dodge wrote: > +1 to improving the signal-to-noise ratio > >> On 29 Sep, 2017, at 11:07, Jason Huynh wrote: >> >> GEODE-3280 >

Re: JPF breaking IntelliJ build?

2017-09-27 Thread Dan Smith
Hmm, it works for me. Looks like the dependencies of your geode-core_jpfTest module are broken - you should see dependencies on geode-concurrenc-test_main, and junit. How did you configure intellij? Did you just import from the gradle model? -Dan On Wed, Sep 27, 2017 at 1:48 PM, Galen O'Sullivan

Re: New client/server protocol - seeking feedback

2017-09-25 Thread Dan Smith
on, > including the details you mention. We'd also like to have a simple client > implemented in Go to go along with the "how-to". > > > -Brian > > On Thu, Sep 21, 2017 at 10:48 AM, Dan Smith wrote: > > > I'm curious about few things I see in the .proto file

Re: []DISCUSS] Using package names to identify public API's

2017-09-22 Thread Dan Smith
I found bug GEODE-3473 which was created based on this discussion. I marked that as requiring a fix for 1.3.0, so we don't ship these protobuf related internal classes as part of the public API. -Dan On Sat, Aug 12, 2017 at 9:42 AM, John Blum wrote: > On the flip side of internal vs. external A

Re: Please don't commit broken javadocs

2017-09-22 Thread Dan Smith
Regarding the duplicate javadocs from geode-assembly - geode-assembly builds the public javadocs for all of geode. Each module also builds it's own javadocs which includes all of the internal methods. That does point out that everything in geode-protobuf seems to be in non internal packages, meani

Re: New client/server protocol - seeking feedback

2017-09-21 Thread Dan Smith
straightforward, but we'll have a more user-friendly > write-up soon. > > > On Sep 15, 2017 5:27 PM, "Dan Smith" wrote: > > What's the best place to look for more details on the specific protocol for > the v1.0 messages? The other pages on https://cwiki.apach

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

2017-09-21 Thread Dan Smith
; > >> code > > > > >> to see if index is updated before the event/change is > saved/injected > > > > into > > > > >> the region...It looks like the index update are happening after > the > > > > region > > > > >

Re: [VOTE] Apache Geode release - v1.2.1 RC4

2017-09-18 Thread Dan Smith
+1 The jenkins release build was green and release check passed as well - https://github.com/upthewaterspout/geode-release-check -Dan On Sat, Sep 16, 2017 at 8:01 AM, Anthony Baker wrote: > This is the fourth release candidate for Apache Geode, version 1.2.1. > Thanks to all the community memb

Re: New client/server protocol - seeking feedback

2017-09-15 Thread Dan Smith
What's the best place to look for more details on the specific protocol for the v1.0 messages? The other pages on https://cwiki.apache.org/ confluence/display/GEODE/New+Client+Server+Protocol? Or directly in the code somewhere? -Dan On Fri, Sep 15, 2017 at 11:15 AM, Brian Baynes wrote: > Greeti

[DISCUSS] Framework for concurrency tests

2017-09-15 Thread Dan Smith
Hi Geode devs, I've been messing around with an open source tool called Java Pathfinder for writing tests of multithreaded code. Java Pathfinder is a special JVM which among other things tries to execute your code using all possible thread interleavings. I'd like to propose two things: 1) We int

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

2017-09-14 Thread Dan Smith
t;> one errors out, will they all be marked invalid? > > > >>>>> > > > >>>>> *Answer:* Only the third will be marked invalid and only the > third > > > >> one > > > >>>> will > > > >>>>

Re: [VOTE] Apache Geode release - v1.2.1 RC3

2017-09-13 Thread Dan Smith
o. > > Anthony > > > On Sep 13, 2017, at 10:22 AM, Dan Smith wrote: > > > > The jenkins build had failures for this revision. They look like they > might > > be environmental issues but I don't think we should actually ship this > > unless we have a green bu

Re: Flaky failures in Geode Nightly Build

2017-09-13 Thread Dan Smith
Please don't mark Jetty9CachingClientServerTest as flaky. We need to look into why it failed and fix it. There are a bunch of other tests that are using the same framework, so marking this one test as flaky isn't going to help. -Dan On Wed, Sep 13, 2017 at 10:07 AM, Jared Stewart wrote: > I wou

Re: [VOTE] Apache Geode release - v1.2.1 RC3

2017-09-13 Thread Dan Smith
The jenkins build had failures for this revision. They look like they might be environmental issues but I don't think we should actually ship this unless we have a green build. Can someone kick the pipeline to restart the build? https://builds.apache.org/blue/organizations/jenkins/Geode-release/de

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

2017-09-08 Thread Dan Smith
+1 -Dan On Thu, Sep 7, 2017 at 9:14 PM, Nabarun Nag wrote: > *Proposal:* > * Index interface will include an API - isValid() which will return true if > the index is still valid / uncorrupted, else will return false if it > corrupted / invalid. > * gfsh command "list index" will have one more c

Re: Missing Gitbox activation email

2017-09-06 Thread Dan Smith
If you are stuck on 3rd step (MFA Status) make you have added your github username on id.apache.org *and* that you have accepted the invitation to join the apache group on github. You should see an apache feather listed underneath your organizations on github. -Dan On Wed, Sep 6, 2017 at 4:08 PM

Re: fix geode-old-versions downloads with a clean build?

2017-09-06 Thread Dan Smith
I submitted a PR to try to cache the old version somewhere else but people didn't like the approach I took so I closed it. If someone wants to take this up that would be great, I'm don't really have time to mess with it right now. https://github.com/apache/geode/pull/733 -Dan On Wed, Sep 6, 2017

Re: [Discuss] Use of -1 as Infinite/All for retry related functions...

2017-08-31 Thread Dan Smith
; see doing that if the server has died but otherwise it just overloads the > servers. > > > > > On 8/30/17 8:36 PM, Dan Smith wrote: > >> In general, I think we need making the configuration of geode less >> complex, >> not more. >> >> As far as re

Re: [Discuss] Use of -1 as Infinite/All for retry related functions...

2017-08-30 Thread Dan Smith
In general, I think we need making the configuration of geode less complex, not more. As far as retry-attempts goes, maybe the best thing to do is to get rid of it. The P2P layer has no such concept. I don't think users should really have to care about how many servers an operation is attempted ag

Re: eviction files

2017-08-24 Thread Dan Smith
Hi Roi, I would probably start from NewLRUClockHand. That's the class that maintains the actual state of what entries are least recently used. You can follow the callers of getLRUEntry back to see how it's being used. One key place is AbstractLRUMap.lruUpdateCallback, which gets called after upda

Re: Adding parallel import/export of snapshots to gfsh

2017-08-22 Thread Dan Smith
I don't really like the idea of adding a separate command. It really is the same command - you just want to have the parallel flag interact with other options. A separate command would be more confusing for users, and more of a maintenance issue as we add more options to export. Having a --path th

Re: Tombstones

2017-08-22 Thread Dan Smith
I believe tombstones are not included in those operations, so you won't see them. -Dan On Tue, Aug 22, 2017 at 10:56 AM, Michael Stolz wrote: > Do tombstones show up in operations that indicate presence of keys like > Region.keyset(), containsKey(key), etc? > They probably shouldn't. > > -- > M

Re: Gitbox enables the full GitHub workflow

2017-08-22 Thread Dan Smith
+1 - Let's do it! -Dan On Tue, Aug 22, 2017 at 10:58 AM, Dave Barnes wrote: > +1 > I'm interested to see how it works for geode-site, which has a sorta quirky > build/branch structure. > > On Tue, Aug 22, 2017 at 10:49 AM, Jared Stewart > wrote: > > > +1 for moving the other repos to Gitbox >

Re: geode-old-versions dependencies is slooooowwww

2017-08-21 Thread Dan Smith
On Mon, Aug 21, 2017 at 3:14 PM, Patrick Rhomberg wrote: > I am also in favor of skipping the download, but now I'm failing my build > Yes, but the important part is that it didn't download anything :) Sorry about that, it looks like I screwed something up. Can you pull and try again? -Dan

Re: geode-old-versions dependencies is slooooowwww

2017-08-21 Thread Dan Smith
Hi Kirk, I just pushed a fix for this. It should only download geode.1.2.0 the first time you build. -Dan On Mon, Aug 21, 2017 at 2:19 PM, Kirk Lund wrote: > Why does "geode-old-versions" download previous versions every time I > rebuild from the command-line? And why is it s slow? Can't w

Re: [Discuss] Compatibility with Apache Xerces

2017-08-18 Thread Dan Smith
> Since the oracle parser is always going to be there I don't see any harm in doing that. That's not true if we're running on non-oracle JDKs, right? I remember a while back someone was trying to run geode on IBMs JDK and having issues - maybe even this same whitespace problem? I think it this fi

Native commit messages going to dev instead of commit list

2017-08-17 Thread Dan Smith
ured when the main repo was. It > is not something I can change even though we are not a mirror site. > > > Mark, do you have karma to change this? > > -Jake > > > On Thu, Aug 17, 2017 at 11:25 AM Dan Smith wrote: > > > I think these messages should be going to

Re: [DISCUSS] authorizing function execution

2017-08-17 Thread Dan Smith
On Thu, Aug 17, 2017 at 12:10 PM, Swapnil Bawaskar wrote: > So, it sounds like if we did #1 above: i.e: > 1) externalize SecurityService so that function author can use it in the > function.execute code to check authorization. > > we could get this to work with lambdas: > ResultCollector rc = get

Re: [geode-native] branch develop updated: GEODE-3455: Update Solaris build images.

2017-08-17 Thread Dan Smith
I think these messages should be going to commits@geode? On Thu, Aug 17, 2017 at 11:15 AM, wrote: > This is an automated email from the ASF dual-hosted git repository. > > jbarrett pushed a commit to branch develop > in repository https://gitbox.apache.org/repos/asf/geode-native.git > > > The fo

Re: [DISCUSS] authorizing function execution

2017-08-17 Thread Dan Smith
my mind would be, if we get to Lambda-based > functions, how do we specify authorization credentials then? Annotations > are great to "static" definition on services, not so great for more > "dynamic" execution of stuff... > > > > On 8/17/17 09:29

Re: [DISCUSS] authorizing function execution

2017-08-17 Thread Dan Smith
I like the annotation idea, especially considering that Jared was talking about adding a @RegisterableFunction annotation as well. maybe something like @ResourcePermission("Data:READ") or something like that? -Dan On Thu, Aug 17, 2017 at 8:18 AM, Michael William Dodge wrote: > What about an ann

Re: PdxInitializationException: PDX registry could not be initialized

2017-08-10 Thread Dan Smith
Hi Aashita, Do you have a stack trace for that PdxInitializationException? Maybe there is a cause that has a bit more information? -Dan On Thu, Aug 10, 2017 at 4:24 PM, Mark Bretl wrote: > + user > > --Mark > > On Thu, Aug 10, 2017 at 4:12 PM, Aashita Sharma < > aashitasha...@yahoo.co.in.inval

Re: [DISCUSS] Improvements to backups

2017-08-10 Thread Dan Smith
+1 this all looks good to me. I think #2 in particular would probably simplify the incremental backup code. For #3, I could have sworn the backups were already going into timestamped directories and nothing got overwritten in an existing backup. If that is not already happening that definitely sho

Re: Geode Module Configuration

2017-08-09 Thread Dan Smith
There is no global configuration for lucene right now. Adding extensions to the cache.xml *looks* like it should be possible. InternalCache implements Extensible which means that theoretically you should be able to call InternalCache.getExtensionPoint.addExtension(yourXmlParser). That whole Extens

Re: Geode Nightly Build with Docker

2017-08-09 Thread Dan Smith
I don't think this is a port conflict issue. I think this is a matter of what our code does when we are running on an ubuntu machine with the default configuration. Ubuntu puts an entry in /etc/hosts mapping 127.0.1.1 to the local hostname. We saw this yesterday and Bruce was going to create a cou

Re: Gitbox enables the full GitHub workflow

2017-08-08 Thread Dan Smith
I'm in favor of switching to gitbox. We're already getting pretty much all contributions from non-committers coming in as pull requests so we might as well make it easier for us to manage things in github. -Dan On Tue, Aug 8, 2017 at 8:57 AM, Kirk Lund wrote: > One thing that's given me trouble

[DRAFT] Geode Board Report August 2017

2017-08-07 Thread Dan Smith
Hi folks, Please review the below draft for our report to the board for this past quarter and offer corrections. As Mark mentioned, we need to submit the draft this Wednesday. Thanks! -Dan - ## Description: - Apache Geode provides a datab

Re: Volunteer Request: Write Draft Board Report For August 2017

2017-08-07 Thread Dan Smith
Hi Mark, I can take this on. I'll try to get a draft out today. -Dan On Fri, Aug 4, 2017 at 12:06 PM, Mark Bretl wrote: > Hi Everyone, > > We are scheduled to report to the Apache board for this months board > meeting on August 16 and need to submit a report by Wednesday, August 9th. > Goal wo

Re: [DISCUSS] Feature branch cleanup

2017-08-03 Thread Dan Smith
On Thu, Aug 3, 2017 at 1:41 PM, Jinmei Liao wrote: > I am not sure why I have so many branches there. I don't remember I created > them. But this is what I get when I tried to delete one of them: > > *$* git fetch origin > > *$* git push origin --delete feature/GEODE-2632-6 > > error: unable to

Re: Need to know complete execution time for geode-core Junit tests

2017-07-25 Thread Dan Smith
What target are you running? It's a bit confusing because there are actually 3 different sets of tests. These times a really rough because I don't have a good run in front of me, but this should give you an idea. "True" unit tests, runs in 1-2 minutes like Jens Said: ./gradlew geode-core:test Sin

Re: Proposal: Lucene indexing/searching for nested objects

2017-07-20 Thread Dan Smith
On Thu, Jul 20, 2017 at 10:57 AM, Jacob Barrett wrote: > I really feel like an annotation would make the most sense. How likely is > it that the object could not be annotated or the serializer for the object > is not tightly coupled with the object? Having to map objects to > serializers external

Re: Proposal: Lucene indexing/searching for nested objects

2017-07-20 Thread Dan Smith
also accept a serializer, like this create lucene index --serializer=com.mycompany.MySerializer If there are no objections I'll update the proposal. -Dan On Tue, Jul 18, 2017 at 10:38 AM, Dan Smith wrote: > I think this LuceneSerializer API needs a slight tweak. In order to > implem

Re: Proposal: Lucene indexing/searching for nested objects

2017-07-18 Thread Dan Smith
LuceneSerializer { Collection toDocuments(Object value, *LuceneIndex index*); } On Thu, Jul 13, 2017 at 2:19 PM, Dan Smith wrote: > On Thu, Jul 13, 2017 at 11:26 AM, Jacob Barrett > wrote: > >> Collections are really tough in Lucene because you have to flatten the >> document. I str

Re: need information about SerialGatewaySenderQueue/ParallelGatewaySenderQueue Clear

2017-07-17 Thread Dan Smith
Hi Dinesh, I think we probably just never got around to adding a clear. I think you could probably clear your queues just stop stopping and starting the gateway sender, which might be the easiest thing to do here. Regarding your code, for your parallel queue are you doing that inside of a functio

Re: Can someone grant me wiki edit privileges

2017-07-17 Thread Dan Smith
Done! -Dan On Mon, Jul 17, 2017 at 10:50 AM, Brian Rowe wrote: > Can someone with the relevant karma please grant me edit permissions on > cwiki.apache.org/confluence. My username there is browe. > > Thanks, > -Brian >

Re: Review Request 60875: GEODE-2654: Backups can capture different members from different points in time

2017-07-14 Thread Dan Smith
> On July 14, 2017, 9 p.m., Dan Smith wrote: > > I like where you are going with this and I think this will make the backup > > a lot more consistent. I think what you are doing with the oplogs looks > > good. However, the changes made to DiskInitFile don't look lik

Re: Review Request 60875: GEODE-2654: Backups can capture different members from different points in time

2017-07-14 Thread Dan Smith
der one lock but getting the value of drMapByName under a different lock. - Dan Smith On July 14, 2017, 5:40 p.m., Lynn Gallinat wrote: > > --- > This is an automatically generated e-mail. To repl

Re: Proposal: Lucene indexing/searching for nested objects

2017-07-13 Thread Dan Smith
On Thu, Jul 13, 2017 at 11:26 AM, Jacob Barrett wrote: > Collections are really tough in Lucene because you have to flatten the > document. I struggled against it for some time on a project a few years ago > and ultimately decided to index the relationships separately and then merge > the results

Re: Proposal: Lucene indexing/searching for nested objects

2017-07-13 Thread Dan Smith
+1 Looks good. I think we should consider adding support for collections as well, but that doesn't have to be in the first cut. -Dan On Wed, Jul 12, 2017 at 10:37 AM, Diane Hardman wrote: > The Geode 1.2.0 release includes Lucene text search fully integrated and > tested (no longer experimental

Re: [VOTE] Apache Geode release - v1.2.0 RC2

2017-07-12 Thread Dan Smith
I created GEODE-3199 for the gpg prompting issue with geode-examples. -Dan On Wed, Jul 12, 2017 at 2:04 PM, Dan Smith wrote: > Travis was actually trying to tell us something was wrong with the > geode-examples on the release branch as well: > > https://travis-ci.org/apache/ge

Re: [VOTE] Apache Geode release - v1.2.0 RC2

2017-07-12 Thread Dan Smith
See https://cwiki.apache.org/confluence/display/GEODE/Release+Steps FAILURE: Build failed with an exception. -Dan On Wed, Jul 12, 2017 at 1:28 PM, Dan Smith wrote: > +0 > > I ran geode-release-check > <https://github.com/upthewaterspout/geode-release-check> against this >

Re: [VOTE] Apache Geode release - v1.2.0 RC2

2017-07-12 Thread Dan Smith
+0 I ran geode-release-check against this project. It all looks good except that the md5sum and sha256 sums tasks don't pass because the .md5 and .sha256 files are in the wrong format. I created GEODE-3196. I don't think that should hold up

Re: Requesting edit permissions for Geode tickets

2017-07-12 Thread Dan Smith
I don't see that user. I think you probably need to create an account on the wiki first. -Dan On Wed, Jul 12, 2017 at 12:09 PM, Fred Krone wrote: > I don't think I have access. I thought I did. fkrone ? > > On Wed, Jul 12, 2017 at 2:36 PM, Dan Smith wrote: > > >

Re: Requesting wiki edit privileges

2017-07-12 Thread Dan Smith
I added you, you should have access now. Thanks! -Dan On Wed, Jul 12, 2017 at 10:12 AM, Diane Hardman wrote: > Will someone please give me edit privileges for Geode wiki pages? > > Thanks! > Diane Hardman >

Re: Requesting edit permissions for Geode tickets

2017-07-12 Thread Dan Smith
Hi Fred, What's your username on the wiki? -Dan On Wed, Jul 12, 2017 at 11:32 AM, Fred Krone wrote: > Could someone give me karma on the wiki? > > Thanks, > -Fred > > On Thu, Mar 30, 2017 at 1:20 PM, Kirk Lund wrote: > > > I gave you permissions for Jira. Someone else still needs to give you

Re: Review Request 60664: GEODE-3153 Client receives duplicate events during rolling upgrade

2017-07-05 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60664/#review179694 --- Ship it! Ship It! - Dan Smith On July 5, 2017, 7:01 p.m

Re: Geode "minor minor" versions

2017-07-05 Thread Dan Smith
Hi Roi, > Is there any plan to release a 1.1.2, or a 1.2.1, or will this be simply > an "ad hoc" situation, such that only an important bug fix will trigger > it... I understand now that this is the case. > That's exactly it. We'll only release a newer minor revision of older version of geode if

Re: Build failed in Jenkins: Geode-release #72

2017-07-03 Thread Dan Smith
Looks like compilation of the release branch was broken by these changes. I went ahead and fixed the compilation error since we are trying to get this release out - I just made ProcessManager match the contents on develop, which I think was the intention. commit 30e2eb2080afff3af2f5226a412259c3a53

<    1   2   3   4   5   6   7   8   9   10   >