[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #198 was SUCCESSFUL (with 1222 tests)

--- Spring Data GemFire > Nightly-ApacheGeode > #198 was successful. --- Scheduled 1226 tests in total. https://build.spring.io/browse/SGF-NAG-198/ -- This

Re: [VOTE] Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1

I do see it as something we should address before graduation but not necessarily needed for a release. So unless the mentors think otherwise we can address that even before a final 1.0 in between the milestones and RCs. Sent from my iPhone > On Jan 22, 2016, at 5:53 PM, Roman Shaposhnik wr

Fwd: [VIDEO] Geode Clubhouse - January 19th

Hi all, Here's the video from the virtual Geode Clubhouse meeting that was held on January 19th, 2016. A huge thanks goes to Edin Zulich for presenting on The New Geode Off-heap Memory Storage Feature, Upcoming in 1.0 Alpha Release! Link to video: http://bit.ly/geodeclubhouse2 Thanks, Karen

Re: [VOTE] Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1

On Fri, Jan 22, 2016 at 5:51 PM, Anthony Baker wrote: > I suggest we extend the vote through Monday January 25 at 5pm PST to > clarify this question and any issues related to the binary NOTICE file. Sure. But if you could state your opinion on how the question I asked should be answered that'd be

Re: [VOTE] Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1

I suggest we extend the vote through Monday January 25 at 5pm PST to clarify this question and any issues related to the binary NOTICE file. Anthony > On Jan 22, 2016, at 5:03 PM, Roman Shaposhnik wrote: > > On Fri, Jan 22, 2016 at 4:32 PM, Anthony Baker wrote: >> Originally, gemfire-json an

Review Request 42679: GEODE-832: Modifying the dunit framework to allow lambda expressions

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42679/ --- Review request for geode, Jens Deppe and Kirk Lund. Repository: geode Descrip

Re: Please Join us for the Inagural Geode Summit - early bird pricing through Jan 31st!

Here's a sandbox site showing my changes: http://ec2-52-33-216-127.us-west-2.compute.amazonaws.com/community/ On Fri, Jan 22, 2016 at 4:50 PM, William Markito wrote: > Hey Dave, > > Apache mailing lists doesn't allow attachments so you might want to host it > somewhere else like imgur and post

Re: [VOTE] Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1

On Fri, Jan 22, 2016 at 4:32 PM, Anthony Baker wrote: > Originally, gemfire-json and gemfire-joptsimple were bundled to fix bugs > present in those libs. > There’s an intent to move away from the JSON parser entirely in GEODE-629. > Attached to that bug is a diff of the changes made to the origin

Re: Please Join us for the Inagural Geode Summit - early bird pricing through Jan 31st!

Hey Dave, Apache mailing lists doesn't allow attachments so you might want to host it somewhere else like imgur and post the link here. It might also be interesting to submit a PR for review as an alternative to review and might you even be able to attach your screenshot there, but YMMV. Thanks!

Re: [VOTE] Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1

Originally, gemfire-json and gemfire-joptsimple were bundled to fix bugs present in those libs. There’s an intent to move away from the JSON parser entirely in GEODE-629. Attached to that bug is a diff of the changes made to the original source. For the joptsimple library, I’ve filed a new JI

Re: Please Join us for the Inagural Geode Summit - early bird pricing through Jan 31st!

I don't have a handy place to install the full site, but here's a screenshot of the events panel. If/when it looks OK to you, I'll submit a pull request to have the changes put in place. -Dave On Fri, Jan 22, 2016 at 1:25 PM, Gregory Chase wrote: > Seattle JUG - Feb 16 http://www.seajug.org/ >

[GitHub] incubator-geode pull request: GEODE-12: license file update

Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/75 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

Re: [VOTE] Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1

This afternoon I did build successfully from the source distribution after installing gradle from gradle.org. Updating the documentation looks like just adding the step to install gradle and specifying gradle in place of gradlew everywhere? Ken > On Jan 22, 2016, at 1:56 PM, Anthony Baker wro

Re: [VOTE] Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1

On gradlew: we can’t include those in the source distribution since it requires a binary jar file. There is an alternate implementation used in the bigtop project that avoid this issue. See comment on GEODE-826: https://issues.apache.org/jira/browse/GEODE-826?page=com.atlassian.jira.plugin.sy

Re: Review Request 42669: GEODE_634 - suspect string causes CI failure

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42669/#review115938 --- Ship it! Ship It! - Hitesh Khamesra On Jan. 22, 2016, 7:44 p

Re: [VOTE] Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1

Thanks Ken for your feedback! 1. I too noticed it though I was able to run gradle directly from the src directory. I vaguely recollect discussion to exclude gradle binaries but not sure if build scripts fall in the same category. In any case, the documentation should be sync'ed with the dist.

Re: Pulse UI merged to develop

Ok, I think I know why. Somehow at one point, another Ignore class was imported, this jdk.nashorn.internal.ir.annotations.Ignore. That's why Junit is not honoring it. I've checked in a fix, waiting to be pulled as well. Good find! On Fri, Jan 22, 2016 at 1:03 PM, Jinmei Liao wrote: > Yes, I ran

Re: Please Join us for the Inagural Geode Summit - early bird pricing through Jan 31st!

Seattle JUG - Feb 16 http://www.seajug.org/ On Fri, Jan 22, 2016 at 1:20 PM, Gregory Chase wrote: > Sunday, Jan 31st - FOSDEM > Mar 15 - Geode Clubhouse > > The Feb Clubhouse is still being set up. > > On Fri, Jan 22, 2016 at 1:17 PM, Gregory Chase wrote: > >> Oops, can't forget FOSDEM >> >> An

Re: Please Join us for the Inagural Geode Summit - early bird pricing through Jan 31st!

Sunday, Jan 31st - FOSDEM Mar 15 - Geode Clubhouse The Feb Clubhouse is still being set up. On Fri, Jan 22, 2016 at 1:17 PM, Gregory Chase wrote: > Oops, can't forget FOSDEM > > And more events get added here over time: > https://calendar.google.com/calendar/embed?src=pivotal.io_u8kgvuahjkboh1g

Re: Please Join us for the Inagural Geode Summit - early bird pricing through Jan 31st!

Oops, can't forget FOSDEM And more events get added here over time: https://calendar.google.com/calendar/embed?src=pivotal.io_u8kgvuahjkboh1gnfhv5ts2v9c%40group.calendar.google.com&ctz=America/Los_Angeles On Fri, Jan 22, 2016 at 1:16 PM, Gregory Chase wrote: > + Karen > > Lets just keep the dat

Re: Please Join us for the Inagural Geode Summit - early bird pricing through Jan 31st!

+ Karen Lets just keep the dates Fresh. @Karen -> any more set dates we can add? On Fri, Jan 22, 2016 at 1:13 PM, Dave Barnes wrote: > I can make this change, Greg. > Shall I keep the old postings from 2015 events or start fresh? > > On Fri, Jan 22, 2016 at 12:08 PM, Gregory Chase wrote: > >

Re: Please Join us for the Inagural Geode Summit - early bird pricing through Jan 31st!

I can make this change, Greg. Shall I keep the old postings from 2015 events or start fresh? On Fri, Jan 22, 2016 at 12:08 PM, Gregory Chase wrote: > You are correct - needs to be updated. Current website setup is a bit > challenging for non-developers to update. > > On Fri, Jan 22, 2016 at 12:

Re: Review Request 42664: GEODE-825: A client with subscriptions enabled receives an additional event when invoking get with a CacheLoader

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42664/#review115927 --- Ship it! Ship It! - Dan Smith On Jan. 22, 2016, 6:31 p.m., B

Re: Pulse UI merged to develop

Yes, I ran the uiTest with 100% success rate that only includes those tests in PulseAutomatedTest, but you are right, after pulled the develop, those tests do get run. I'll take a look at exactly what was the cause. Thanks! On Fri, Jan 22, 2016 at 12:41 PM, Nitin Lamba wrote: > I meant I ran './

Re: Pulse UI merged to develop

I meant I ran './gradlew uiTest' in my setup for UI testing and got those errors. I do see those two lines on the PulseUI test so source code is up to date. Were you able to run the UI tests completely without errors? In other classes, I had to add @Ignore for every test and have not used it fo

Re: Review Request 42659: GEODE-12 Update Selenium for latest FireFox (v43.x); Fix tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42659/#review115918 --- Ship it! Ship It! - Dick Cavender On Jan. 22, 2016, 5:06 p.m.,

Re: Please Join us for the Inagural Geode Summit - early bird pricing through Jan 31st!

You are correct - needs to be updated. Current website setup is a bit challenging for non-developers to update. On Fri, Jan 22, 2016 at 12:05 PM, Mark Bretl wrote: > Hi Greg, > > I don't see this event on the community site, > http://geode.incubator.apache.org/community/#events, would you be ab

Docker image updated with RC1

Not advertising this outside of the development community yet, but I've just updated our docker image with our RC1. For example, to start gfsh: docker run -it apachegeode/geode:1.0.0-incubating.M1.RC1 gfsh As the process move forward I'll update the instructions as well but feel free to try it ou

Re: Please Join us for the Inagural Geode Summit - early bird pricing through Jan 31st!

Hi Greg, I don't see this event on the community site, http://geode.incubator.apache.org/community/#events, would you be able to add it? --Mark On Thu, Jan 14, 2016 at 9:54 AM, Gregory Chase wrote: > Pivotal Software is very excited to host this event for the Apache Geode > (incubating) commun

Re: Review Request 42664: GEODE-825: A client with subscriptions enabled receives an additional event when invoking get with a CacheLoader

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42664/#review115916 --- Ship it! - Bruce Schuchardt On Jan. 22, 2016, 6:31 p.m., Barry O

Re: Review Request 42655: GEODE-830: Allow transitive test dependencies

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42655/#review115914 --- Ship it! Remember to remove any properties declared in dependency-

Review Request 42669: GEODE_634 - suspect string causes CI failure

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42669/ --- Review request for geode and Hitesh Khamesra. Repository: geode Description -

Re: Review Request 41836: GEODE-719: Add error logs while cache.xml processing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41836/ --- (Updated Jan. 22, 2016, 7:22 p.m.) Review request for geode and Darrel Schneide

Re: Build time and time per task

Pretty sure it's the addition of javadocs that has changed the time recently. -Dan On Fri, Jan 22, 2016 at 10:38 AM, Barry Oglesby wrote: > I've used --profile before. It creates a nice report > in build/reports/profile like: > > :pivotalgf-assembly 33.731s (total) > :pivotalgf-assembly:srcDist

Re: Pulse UI merged to develop

When you simply run "test", it shouldn't even run those uiTest at all since it's excluded from the test tasks. Do open up your PulseUITest.java and see if it has those lines on top of the class: @Ignore @Category(UITest.class) If not, then your source is not up to date? On Fri, Jan 22, 2016 at

Re: [VOTE] Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1

A JIRA was filed on 1/21 by Dick Cavender on the gradlew issue, https://issues.apache.org/jira/browse/GEODE-826 The JDK version in README.md was fixed earlier. It’s on the develop branch, but not release/1.0.0-incubating.M1. Are these issues serious enough to re-spin the release builds? Becaus

Review Request 42665: Move dunit to com.gemstone.gemfire.test.dunit

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42665/ --- Review request for geode, Jens Deppe and Jinmei Liao. Bugs: GEODE-819 https

Review Request 42664: GEODE-825: A client with subscriptions enabled receives an additional event when invoking get with a CacheLoader

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42664/ --- Review request for geode, Bruce Schuchardt and Dan Smith. Repository: geode D

Re: Build time and time per task

I've used --profile before. It creates a nice report in build/reports/profile like: :pivotalgf-assembly 33.731s (total) :pivotalgf-assembly:srcDistTar 14.330s :pivotalgf-assembly:distTar 7.054s :pivotalgf-assembly:distZip 6.909s :pivotalgf-assembly:srcDistZip 4.654s :pivotalgf-assembly:compileTest

Build time and time per task

The build has become painful. Is there an argument for gradle to show how much time each task takes? $ ./gradlew -Dskip.tests=true clean build :pivotalgf-assembly:processTestResources UP-TO-DATE :pivotalgf-assembly:testClasses :pivotalgf-assembly:checkMissedTests SKIPPED :pivotalgf-assembly:test

Re: Review Request 42659: GEODE-12 Update Selenium for latest FireFox (v43.x); Fix tests

> On Jan. 22, 2016, 5:35 p.m., Jinmei Liao wrote: > > gemfire-pulse/build.gradle, line 155 > > > > > > The tests don't depend on them anymore. OK. I'll discard this patch. - Nitin ---

Re: Pulse UI merged to develop

Response inline... From: Jinmei Liao Sent: Friday, January 22, 2016 9:34 AM To: dev@geode.incubator.apache.org Subject: Re: Pulse UI merged to develop Nitin, thanks for the review. Good point on updating the Selenium version. As to pulling the version num

[GitHub] incubator-geode pull request: GEODE-12: license file update

GitHub user jinmeiliao opened a pull request: https://github.com/apache/incubator-geode/pull/75 GEODE-12: license file update You can merge this pull request into a Git repository by running: $ git pull https://github.com/jinmeiliao/incubator-geode feature/GEODE-12 Alternativ

Re: Review Request 42659: GEODE-12 Update Selenium for latest FireFox (v43.x); Fix tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42659/#review115874 --- gemfire-pulse/build.gradle (line 155)

Re: Pulse UI merged to develop

Nitin, thanks for the review. Good point on updating the Selenium version. As to pulling the version numbers into the dependency property file, let's wait on that since there is an effort going on to not include all the transitive dependancies in our build script. Once that's done, it would be a lo

Re: Pulse UI merged to develop

Thanks Jinmei. I tried running Pulse UI tests but ran into few issues, listed below: 1. Firefox version has been updated so Selenium jars also requires a new version 2. During the process of cleaning-up build.gradle, setting-up test environmental variables was removed I submitted a review reque

Review Request 42659: GEODE-12 Update Selenium for latest FireFox (v43.x); Fix tests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42659/ --- Review request for geode, Anthony Baker, Dick Cavender, Jens Deppe, Jinmei Liao,

Re: Review Request 42655: GEODE-830: Allow transitive test dependencies

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42655/#review115862 --- Ship it! Ship It! - Dan Smith On Jan. 22, 2016, 4:28 p.m., Jens

Re: Review Request 42655: GEODE-830: Allow transitive test dependencies

> On Jan. 22, 2016, 4:32 p.m., Anthony Baker wrote: > > The change looks good. Were you also going to remove the declared > > transitive deps (such as hamcrest) from the build files? Yes, I can do that. - Jens --- This is an automatic

Re: Review Request 42655: GEODE-830: Allow transitive test dependencies

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42655/#review115853 --- The change looks good. Were you also going to remove the declared

Review Request 42655: GEODE-830: Allow transitive test dependencies

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42655/ --- Review request for geode, Anthony Baker and Dan Smith. Repository: geode Desc

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

This should be fixed; at least the part that prevented Pulse archives from being published. --J On Fri, Jan 22, 2016 at 2:10 AM, Apache Jenkins Server < jenk...@builds.apache.org> wrote: > See > > Changes: > > [upthewaterspout] GEODE-716:

Build failed in Jenkins: Geode-nightly #354

See Changes: [upthewaterspout] GEODE-716: Getting the bucket creation lock when setting the CacheLoader [upthewaterspout] GEODE-681: Measuring event queue size without using the stat for gfsh [bschuchardt] GEODE-369: [jdeppe] GEODE-74