Review Request 53335: GEODE-2028: Fix license issues from 1.0.0-incubating release

2016-10-31 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53335/ --- Review request for geode and Dick Cavender. Repository: geode Description ---

Re: Review Request 53331: GEODE-2047 Document change to enable-network-partition-detection

2016-10-31 Thread Joey McAllister
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53331/#review154352 --- geode-docs/managing/troubleshooting/recovering_conflicting_data_e

Re: Review Request 53108: GEODE-2012: always write stat types to archive

2016-10-31 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53108/#review154350 --- geode-core/src/main/java/org/apache/geode/internal/logging/Manage

Review Request 53331: GEODE-2047 Document change to enable-network-partition-detection

2016-10-31 Thread Karen Miller
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53331/ --- Review request for geode, Bruce Schuchardt, Dave Barnes, and Joey McAllister. R

Re: Review Request 53108: GEODE-2012: always write stat types to archive

2016-10-31 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53108/ --- (Updated Oct. 31, 2016, 11:54 p.m.) Review request for geode, Anthony Baker, Da

Re: Review Request 53108: GEODE-2012: always write stat types to archive

2016-10-31 Thread Kirk Lund
> On Oct. 31, 2016, 8:34 p.m., Darrel Schneider wrote: > > geode-core/src/main/java/org/apache/geode/internal/statistics/StatMonitorHandler.java, > > line 93 > > > > > > only the null check is needed here. > > Sh

[GitHub] incubator-geode pull request #277: GEODE-1098: Fix gfsh.bat error with defau...

2016-10-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/277 --- 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 featur

Re: Docker image for Geode 1.0.0-incubating - Please evaluate

2016-10-31 Thread William Markito Oliveira
Done. apachegeode/geode:latest On Mon, Oct 31, 2016 at 12:42 PM, William Markito Oliveira < william.mark...@gmail.com> wrote: > Sounds cool. That's a great ideia! > > Sent from my iPhone > > > On Oct 31, 2016, at 12:41 PM, Anthony Baker wrote: > > > > Nice! What about tagging it with 'latest'?

Re: [DISCUSS] Graduation

2016-10-31 Thread Jianxia Chen
+1 On Mon, Oct 31, 2016 at 2:47 PM, Anilkumar Gingade wrote: > +1 > > On Fri, Oct 28, 2016 at 10:23 PM, Avinash Dongre > wrote: > > > +1 > > > > On Sat, Oct 29, 2016 at 8:40 AM, yang theseus > > wrote: > > > > > +1! > > > > > > 2016-10-28 7:07 GMT+08:00 Nitin Lamba : > > > > > > > +1! > > > >

Re: Review Request 53311: GEODE-2045 Improve docs of region shortcuts

2016-10-31 Thread Joey McAllister
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53311/#review154324 --- Ship it! Ship It! - Joey McAllister On Oct. 31, 2016, 7:28 p

Re: [DISCUSS] Graduation

2016-10-31 Thread Anilkumar Gingade
+1 On Fri, Oct 28, 2016 at 10:23 PM, Avinash Dongre wrote: > +1 > > On Sat, Oct 29, 2016 at 8:40 AM, yang theseus > wrote: > > > +1! > > > > 2016-10-28 7:07 GMT+08:00 Nitin Lamba : > > > > > +1! > > > > > > On Wed, Oct 26, 2016 at 12:36 PM, Nabarun Nag wrote: > > > > > > > +1 > > > > On Wed, O

Re: Review Request 53108: GEODE-2012: always write stat types to archive

2016-10-31 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53108/#review154312 --- geode-core/src/main/java/org/apache/geode/internal/io/RollingFile

Re: Review Request 53108: GEODE-2012: always write stat types to archive

2016-10-31 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53108/#review154307 --- geode-core/src/test/java/org/apache/geode/internal/statistics/Sta

Re: Review Request 53311: GEODE-2045 Improve docs of region shortcuts

2016-10-31 Thread Dave Barnes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53311/#review154308 --- Ship it! Ship It! - Dave Barnes On Oct. 31, 2016, 7:28 p.m.,

Re: Review Request 53108: GEODE-2012: always write stat types to archive

2016-10-31 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53108/ --- (Updated Oct. 31, 2016, 7:56 p.m.) Review request for geode, Anthony Baker, Dar

Re: Docker image for Geode 1.0.0-incubating - Please evaluate

2016-10-31 Thread William Markito Oliveira
Sounds cool. That's a great ideia! Sent from my iPhone > On Oct 31, 2016, at 12:41 PM, Anthony Baker wrote: > > Nice! What about tagging it with 'latest'? > > Anthony > >> On Oct 31, 2016, at 12:13 PM, William Markito Oliveira >> wrote: >> >> I've just updated our Docker image with 1.0.0

Re: Docker image for Geode 1.0.0-incubating - Please evaluate

2016-10-31 Thread Anthony Baker
Nice! What about tagging it with 'latest'? Anthony > On Oct 31, 2016, at 12:13 PM, William Markito Oliveira > wrote: > > I've just updated our Docker image with 1.0.0-incubating bits, please > evaluate. > > There is also a pull request for review updating our Dockerfile to use > 1.0.0-incuba

Review Request 53311: GEODE-2045 Improve docs of region shortcuts

2016-10-31 Thread Karen Miller
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53311/ --- Review request for geode, Dave Barnes and Joey McAllister. Repository: geode

Docker image for Geode 1.0.0-incubating - Please evaluate

2016-10-31 Thread William Markito Oliveira
I've just updated our Docker image with 1.0.0-incubating bits, please evaluate. There is also a pull request for review updating our Dockerfile to use 1.0.0-incubating and OpenJDK instead of Oracle. - https://hub.docker.com/r/apachegeode/geode/tags/ - https://github.com/apache/incubator-geode/pul

Re: Review Request 53274: GEODE-1955: fix flaky tests by properly closing the gfsh instance in ConnectToLocatorSSLDUnitTest

2016-10-31 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53274/#review154301 --- geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshSh

Re: Review Request 53274: GEODE-1955: fix flaky tests by properly closing the gfsh instance in ConnectToLocatorSSLDUnitTest

2016-10-31 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53274/#review154300 --- Use of the Rule should be cleaned up so everything is contained in

[GitHub] incubator-geode issue #275: [GEODE-1871] - Updating and cleaning up Dockerfi...

2016-10-31 Thread markito
Github user markito commented on the issue: https://github.com/apache/incubator-geode/pull/275 Done! --- 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

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

2016-10-31 Thread Anthony Baker
I updated the geode-examples dependency to 1.0.0-incubating. Anthony > On Oct 31, 2016, at 8:54 AM, Apache Jenkins Server > wrote: > > See > > -- > [...truncated 476 lines...] > :geode-common:spotlessJa

Build failed in Jenkins: Geode-nightly #639

2016-10-31 Thread Apache Jenkins Server
See -- [...truncated 476 lines...] :geode-common:spotlessJavaCheck :geode-common:spotlessCheck :geode-common:test :geode-common:check :geode-common:build :geode-common:distributedTest :geode-common:flakyTest

Badge for your apachegeode/geode image

2016-10-31 Thread Liz Rice
We saw your image on Docker Hub at href="https://hub.docker.com/r/apachegeode/geode";>apachegeode/geode, and we thought you might like this badge for your Docker Hub page showing the image contents. The badge shows the number of layers and download size of your Docker image, and links to our si

[GitHub] incubator-geode pull request #:

2016-10-31 Thread vectorijk
Github user vectorijk commented on the pull request: https://github.com/apache/incubator-geode/commit/c1216eb1817df3c6b20afbaff49ba30e76d34b44#commitcomment-19630739 @upthewaterspout Should we update version number to `1.1.0-incubating-SNAPSHOT`? Also, I have a question about publ