Re: TLP Transition Changes Coming

2016-12-02 Thread Mark Bretl
John, Redirects are in place for the incubator website once hosting is moved. Anthony, Lots of questions and I think have most of the answers... - Status of items in [1]: -- All items are completed except for the remaining INFRA transition items. -- Our reporting schedule after the initial three

[GitHub] incubator-geode issue #295: [GEODE-1984] #comment Fix Issue Make GatewaySend...

2016-12-02 Thread davinash
Github user davinash commented on the issue: https://github.com/apache/incubator-geode/pull/295 Hi @metatype Could you please deleting branch GEODE-1984 from master. I am getting error. git flow feature start GEODE-1984 Fatal: Branch 'feature/GEODE-1984' already exists. Pick an

[GitHub] incubator-geode issue #295: [GEODE-1984] #comment Fix Issue Make GatewaySend...

2016-12-02 Thread metatype
Github user metatype commented on the issue: https://github.com/apache/incubator-geode/pull/295 `git push origin --delete feature/GEODE-1984`. You should also confirm that you've deleted the branch from your _local_ repo (`git branch -D feature/GEODE-1984`). Are you still hav

[GitHub] incubator-geode issue #295: [GEODE-1984] #comment Fix Issue Make GatewaySend...

2016-12-02 Thread davinash
Github user davinash commented on the issue: https://github.com/apache/incubator-geode/pull/295 I have no problem with write access and have deleted local branch. But I get following error. ``` git push origin --delete feature/GEODE-1984 error: unable to delete 'feature/GEO

[GitHub] incubator-geode issue #295: [GEODE-1984] #comment Fix Issue Make GatewaySend...

2016-12-02 Thread metatype
Github user metatype commented on the issue: https://github.com/apache/incubator-geode/pull/295 Sorry, I meant to write "I delete the remote branch for you". :-) --- 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 pr

Build failed in Jenkins: Geode-nightly #671

2016-12-02 Thread Apache Jenkins Server
See Changes: [ukohlmeyer] GEODE-1931: Spotless [jiliao] GEODE-2146: deploy should require more elevated privileges than just [bschuchardt] GEODE-2159 add serialVersionUIDs to exception classes not having them [khowe] GEODE-1740: Correc

Review Request 54315: GEODE-2146 document new privileges req'd for deploy

2016-12-02 Thread Karen Miller
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54315/ --- Review request for geode, Dave Barnes, Jinmei Liao, and Joey McAllister. Reposi

Re: Review Request 54315: GEODE-2146 document new privileges req'd for deploy

2016-12-02 Thread Joey McAllister
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54315/#review157771 --- Ship it! Ship It! - Joey McAllister On Dec. 2, 2016, 5:46 p.

Re: Review Request 54315: GEODE-2146 document new privileges req'd for deploy

2016-12-02 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54315/#review157772 --- Ship it! Ship It! - Jinmei Liao On Dec. 2, 2016, 5:46 p.m.,

[GitHub] incubator-geode pull request #302: GEODE-2196: Subclasses of OperationContex...

2016-12-02 Thread kjduling
GitHub user kjduling opened a pull request: https://github.com/apache/incubator-geode/pull/302 GEODE-2196: Subclasses of OperationContext should also be deprecated. precheckin running You can merge this pull request into a Git repository by running: $ git pull https://github.co

Re: Review Request 53857: GEODE-2095: Stabilize flaky tests

2016-12-02 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53857/#review157787 --- Ship it! Ship It! - anilkumar gingade On Nov. 17, 2016, 10:3

Re: TLP Transition Changes Coming

2016-12-02 Thread Roman Shaposhnik
On Fri, Dec 2, 2016 at 12:27 AM, Mark Bretl wrote: > John, Redirects are in place for the incubator website once hosting is > moved. > > Anthony, > > Lots of questions and I think have most of the answers... > > - Status of items in [1]: > -- All items are completed except for the remaining INFRA

Re: TLP Transition Changes Coming

2016-12-02 Thread Anthony Baker
Ah, you’re right. We have multiple INFRA tickets on this in the past so I thought it was working :-) Anthony > On Dec 2, 2016, at 12:27 AM, Mark Bretl wrote: > > -- Note: I do not see a Geode Sonar project on https://analysis.apache.org > , > so we will have to

Re: TLP Transition Changes Coming

2016-12-02 Thread Anthony Baker
+1 Keeping the committer list intact seems like the most sensible course of action. Anthony > On Dec 2, 2016, at 11:33 AM, Roman Shaposhnik wrote: > > Theoretically you could have a re-qualification for committers, but my > recommendation > will simply be to bulk add all of the existing ones t

Re: Review Request 54315: GEODE-2146 document new privileges req'd for deploy

2016-12-02 Thread Dave Barnes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54315/#review157805 --- Ship it! Ship It! - Dave Barnes On Dec. 2, 2016, 5:46 p.m.,

Re: TLP Transition Changes Coming

2016-12-02 Thread Dave Barnes
As I recall, the committer list on the incubator page was far behind the list on the product's web page (under "Community"). On Fri, Dec 2, 2016 at 11:37 AM, Anthony Baker wrote: > +1 > > Keeping the committer list intact seems like the most sensible course of > action. > > Anthony > > > On Dec

Re: TLP Transition Changes Coming

2016-12-02 Thread Mark Bretl
Thanks Roman! Very helpful information. Good reminder, Dave! As long as there are no objections, I will use the Committer list on the Community product page (http://geode.apache.org/community/) to re-add committers to the project. --Mark On Fri, Dec 2, 2016 at 11:53 AM, Dave Barnes wrote: > A

Re: TLP Transition Changes Coming

2016-12-02 Thread Dan Smith
> > As long as there are no objections, I will use the Committer list on the > Community product page (http://geode.apache.org/community/) to re-add > committers to the project. > > +1

Re: TLP Transition Changes Coming

2016-12-02 Thread William Markito Oliveira
+1 as well - Although I'd say that in the near future we should consider cleaning the list a little bit using some criteria... For example, people on that list have NEVER sent a single message to the mailing list, not to mention code contributions, wiki or anything so why keep them ? On Fri,

Re: TLP Transition Changes Coming

2016-12-02 Thread Roman Shaposhnik
On Fri, Dec 2, 2016 at 11:53 AM, Dave Barnes wrote: > As I recall, the committer list on the incubator page was far behind the > list on the product's web page (under "Community"). I believe I updated it to be a pretty close match right before the discussion on graduation. At any rate -- I think

Re: TLP Transition Changes Coming

2016-12-02 Thread Kenneth Howe
+1 > On Dec 2, 2016, at 12:10 PM, Roman Shaposhnik wrote: > > On Fri, Dec 2, 2016 at 11:53 AM, Dave Barnes wrote: >> As I recall, the committer list on the incubator page was far behind the >> list on the product's web page (under "Community"). > > I believe I updated it to be a pretty close ma

Review Request 54321: GEODE-2156 Remove incubating references

2016-12-02 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54321/ --- Review request for geode, Dave Barnes, Dick Cavender, Joey McAllister, Karen Mil

Re: TLP Transition Changes Coming

2016-12-02 Thread Joey McAllister
+1 On Fri, Dec 2, 2016 at 12:12 PM Kenneth Howe wrote: > +1 > > On Dec 2, 2016, at 12:10 PM, Roman Shaposhnik > wrote: > > > > On Fri, Dec 2, 2016 at 11:53 AM, Dave Barnes wrote: > >> As I recall, the committer list on the incubator page was far behind the > >> list on the product's web page (

Re: Review Request 54321: GEODE-2156 Remove incubating references

2016-12-02 Thread Mark Bretl
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54321/#review157819 --- We need to add the new Apache Software Foundation logo to our webs

Broken: apache/incubator-geode#1530 (develop - a46cc97)

2016-12-02 Thread Travis CI
Build Update for apache/incubator-geode - Build: #1530 Status: Broken Duration: 7 minutes and 39 seconds Commit: a46cc97 (develop) Author: Bruce Schuchardt Message: GEODE-2137 client membership IDs contain 17 bytes of useless information There was a problem w

Review Request 54323: GEODE-2163: CqService.serverCqNameCache does not get cleaned up, possible memory leak

2016-12-02 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54323/ --- Review request for geode and Dan Smith. Repository: geode Description ---

Re: Review Request 53857: GEODE-2095: Stabilize flaky tests

2016-12-02 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53857/#review157853 --- Ship it! Ship It! - Darrel Schneider On Nov. 17, 2016, 2:38

Re: Review Request 54323: GEODE-2163: CqService.serverCqNameCache does not get cleaned up, possible memory leak

2016-12-02 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54323/#review157854 --- Fix it, then Ship it! geode-cq/src/main/java/org/apache/geode/

Review Request 54330: GEODE-2179 - Checking for disconnected connection in GatewaySenderMBean

2016-12-02 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54330/ --- Review request for geode and Barry Oglesby. Repository: geode Description ---

Re: Review Request 54330: GEODE-2179 - Checking for disconnected connection in GatewaySenderMBean

2016-12-02 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54330/#review157860 --- Ship it! Ship It! - Jason Huynh On Dec. 2, 2016, 11:48 p.m.,

Review Request 54332: GEODE-2178 - Docs: Update HTTP session management version numbers

2016-12-02 Thread Dave Barnes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54332/ --- Review request for geode, Joey McAllister, Karen Miller, and Dan Smith. Reposit

Fixed: apache/incubator-geode#1534 (develop - a65f85d)

2016-12-02 Thread Travis CI
Build Update for apache/incubator-geode - Build: #1534 Status: Fixed Duration: 7 minutes and 17 seconds Commit: a65f85d (develop) Author: Kirk Lund Message: GEODE-2176: apply FlakyTest category to flaky test This test last failed in Geode-nightly #669 on Nov 3

[GitHub] incubator-geode issue #297: GEODE-734: gfsh export stack-traces should not r...

2016-12-02 Thread deepakddixit
Github user deepakddixit commented on the issue: https://github.com/apache/incubator-geode/pull/297 @metatype and @kirklund I have addressed the review comments. Kindly have a look. precheckin is successful. I am not sure why travis is failed with spotlessJavaCheck, at my box it i

[GitHub] incubator-geode pull request #275: [GEODE-1871] - Updating and cleaning up D...

2016-12-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/275 --- 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