Re: Review Request 44548: GEODE-226: JSON seems to lose time portion on getObject

2016-03-08 Thread Jens Deppe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44548/#review122690 ---

Broken: apache/incubator-geode#85 (feature/GEODE-1050 - 24602c9)

2016-03-08 Thread Travis CI
Build Update for apache/incubator-geode - Build: #85 Status: Broken Duration: 16 minutes and 56 seconds Commit: 24602c9 (feature/GEODE-1050) Author: Kirk Lund Message: Make remaining vars in DistributedTestCase private Fix up one dunit extending JUnit3 and

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

2016-03-08 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #244 was successful. --- Scheduled 1267 tests in total. https://build.spring.io/browse/SGF-NAG-244/ -- This

Re: Review Request 44548: GEODE-226: JSON seems to lose time portion on getObject

2016-03-08 Thread kareem shabazz
Why not have the default as "/mm/dd" so it's universally understood either side of the pond. On Tue, Mar 8, 2016 at 8:47 PM, Jianxia Chen wrote: > > --- > This is an automatically generated e-mail. To reply, visit: >

Review Request 44548: GEODE-226: JSON seems to lose time portion on getObject

2016-03-08 Thread Jianxia Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44548/ --- Review request for geode, Bruce Schuchardt, Hitesh Khamesra, and Udo Kohlmeyer.

Re: Review Request 44540: add junit test for procesChunk of FetchEntriesMessage

2016-03-08 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44540/#review122649 --- Ship it! Ship It! - Dan Smith On March 9, 2016, 12:06 a.m.,

Re: New Committer to Apache Geode: Nitin Lamba

2016-03-08 Thread Mark Bretl
Welcome to the Rock, Nitin! --Mark On Tue, Mar 8, 2016 at 4:49 PM, Nitin Lamba wrote: > Thanks everyone! > > Hi Roman, > > My preferred e-mail id is nlamba > > Best, > Nitin > > Sent from Outlook Mobile > > > > > On Tue, Mar 8, 2016 at 4:48 PM -0800,

Re: New Committer to Apache Geode: Nitin Lamba

2016-03-08 Thread Nitin Lamba
Thanks everyone! Hi Roman, My preferred e-mail id is nlamba Best, Nitin Sent from Outlook Mobile On Tue, Mar 8, 2016 at 4:48 PM -0800, "Anthony Baker" > wrote: Welcome Nitin! > On Mar 8, 2016, at 11:11 AM, Greg Chase

Review Request 44540: add junit test for procesChunk of FetchEntriesMessage

2016-03-08 Thread xiaojian zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44540/ --- Review request for geode and Dan Smith. Bugs: geode-1042

Review Request 44536: GEODE-1038: CI failure: DistributionAdvisorDUnitTest.testGenericAdvice

2016-03-08 Thread Jianxia Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44536/ --- Review request for geode, Bruce Schuchardt, Hitesh Khamesra, and Udo Kohlmeyer.

[GitHub] incubator-geode pull request: GEODE-511 Pulse: Residual 'Gemfire' ...

2016-03-08 Thread jinmeiliao
Github user jinmeiliao commented on the pull request: https://github.com/apache/incubator-geode/pull/111#issuecomment-193958882 +1 --- 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

[GitHub] incubator-geode pull request: GEODE-511 Pulse: Residual 'Gemfire' ...

2016-03-08 Thread upthewaterspout
Github user upthewaterspout commented on the pull request: https://github.com/apache/incubator-geode/pull/111#issuecomment-193948459 +1 --- 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

[GitHub] incubator-geode pull request: GEODE-511 Pulse: Residual 'Gemfire' ...

2016-03-08 Thread nitin-amp
GitHub user nitin-amp opened a pull request: https://github.com/apache/incubator-geode/pull/111 GEODE-511 Pulse: Residual 'Gemfire' changed to 'Geode' Change **Gemfire** in Pulse UI to **Geode** You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-geode pull request: GEODE-511 Residual 'Gemfire' fixed t...

2016-03-08 Thread nitin-amp
Github user nitin-amp commented on the pull request: https://github.com/apache/incubator-geode/pull/110#issuecomment-193929252 Thanks Dan. I'll rebase and re-submit --- 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

[GitHub] incubator-geode pull request: GEODE-511 Residual 'Gemfire' fixed t...

2016-03-08 Thread upthewaterspout
Github user upthewaterspout commented on the pull request: https://github.com/apache/incubator-geode/pull/110#issuecomment-193924869 +1 You should rebase your branch on apache develop. It looks like what happened is that your previous pull request was cherry-picked to develop

[GitHub] incubator-geode pull request: GEODE-511 Residual 'Gemfire' fixed t...

2016-03-08 Thread nitin-amp
Github user nitin-amp closed the pull request at: https://github.com/apache/incubator-geode/pull/110 --- 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

[GitHub] incubator-geode pull request: GEODE-511 Residual 'Gemfire' fixed t...

2016-03-08 Thread scmbuildguy
Github user scmbuildguy commented on the pull request: https://github.com/apache/incubator-geode/pull/110#issuecomment-193926741 +1 --- 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

Re: New Committer to Apache Geode: Nitin Lamba

2016-03-08 Thread Dan Smith
Welcome Nitin! -Dan On Tue, Mar 8, 2016 at 11:11 AM, Greg Chase wrote: > Please welcome *Nitin Lamba* as a new committer to the Apache Geode > project! > > Nitin has already been an avid contributor to the enhancement of Apache > Geode, most notably serving as the release

New Committer to Apache Geode: Nitin Lamba

2016-03-08 Thread Greg Chase
Please welcome *Nitin Lamba* as a new committer to the Apache Geode project! Nitin has already been an avid contributor to the enhancement of Apache Geode, most notably serving as the release manager for the first offiical release of Apache Geode. Being a committer enables easier contribution to

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

2016-03-08 Thread Dan Smith
I filed GEODE-1056 for this bug. I'm not sure this is anything new, it looks like maybe a race condition? -Dan On Mon, Mar 7, 2016 at 11:46 AM, Darrel Schneider wrote: > When I search my emails the first nightly build I see that it started > failing in was: Geode-nightly

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

2016-03-08 Thread Jason Huynh
This test may have failed due to my checkin/change in dependencies. I'll take a look and hope to get a fix in later today... On Tue, Mar 8, 2016 at 4:48 AM, Apache Jenkins Server < jenk...@builds.apache.org> wrote: > See > > Changes: > >

Build failed in Jenkins: Geode-nightly #402

2016-03-08 Thread Apache Jenkins Server
See Changes: [jdeppe] GEODE-36 Interim fix to display correct product name on gfsh cli [jhuynh] GEODE-562: Upgrade commons collections to v3.2.2 [klund] GEODE-949: fix serialization of GemFireSecurityExceptions to improve