Re: TLP Transition Changes Coming

[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 months is February, May,
August, November.
-- Once the private mail list has been confirmed as migrated, then all
non-PMC members will be removed from the list if they have not been already
- Geode Committer List [2]: It is correct as it was submitted by the
resolution (as only PMC members were defined), however, yes I do agree that
even though people were not added to the PMC they should have retained
committer status.
- Common Tasks [3]: I already created INFRA-12938 as the Common Task
ticket. I added those items from your list which I did not have to that
ticket.
-- Note: I do not see a Geode Sonar project on https://analysis.apache.org,
so we will have to request one later.

How would people like to handle re-adding the initial committers? Should it
be done on an as-needed basis? However, I hope we would not need to vote
them into committers again for Geode.

Does anyone have any history or experience with this issue?

Best Regards,

--Mark

On Thu, Dec 1, 2016 at 9:33 PM, Anthony Baker  wrote:

> Also, I think we need additional subtasks on INFRA-12937 for migrating:
>
> - GitHub mirror
> - release folders
> - Travis-CI
> - JIRA
> - ReviewBoard
> - Sonar
>
> See [1] for examples.  LMK if you want me to create the subtasks.
>
> Anthony
>
> [1] https://issues.apache.org/jira/issues/?jql=project%20%
> 3D%20INFRA%20AND%20component%20%3D%20%22TLP%20Admin%22%
> 20ORDER%20BY%20updated%20DESC%2C%20priority%20DESC%2C%20created%20ASC
>
> > On Dec 1, 2016, at 9:09 PM, Anthony Baker  wrote:
> >
> > Thanks Mark!  Can you let us know what has been done so far according to
> [1]?
> >
> > I see that p.a.o [2] has been updated, but I’m curious about the
> committer list.  It looks quite different compared to during incubation.
> Is that correct?
> >
> > Anthony
> >
> > [1] http://incubator.apache.org/guides/graduation.html#new-
> project-hand-over
> > [2] https://people.apache.org/committers-by-project.html#geode
> >
> >> On Dec 1, 2016, at 5:23 PM, Mark Bretl  wrote:
> >>
> >> Hi Everyone,
> >>
> >> I have received notification that the INFRA team will hopefully start
> our
> >> TLP transition tasks either tomorrow, Friday, or Monday.
> >>
> >> These tasks consist of, but not limited to:
> >> mailing list migration
> >> Apache Git URL migration
> >> GitHub URL migration
> >> JIRA/GitHub Bot
> >>
> >> I am sure things might break along the way, so please be patient with
> the
> >> process, and I will try to keep the community up-to-date as possible.
> >>
> >> --Mark
> >
>
>

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

[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 another name.



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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 having problems with 
write access?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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/GEODE-1984': remote ref does not exist
error: failed to push some refs to 
'https://git-wip-us.apache.org/repos/asf/incubator-geode.git'
```


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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 project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Build failed in Jenkins: Geode-nightly #671

[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: Correct potential region inconsistencies with concurrent

[khowe] GEODE-1740: Apply spotless check

[ukohlmeyer] GEODE-1931: Fixed serialization issue with Lambda

[ukohlmeyer] GEODE-1931: Spotless

[dbarnes] GEODE-2129: Docs - clarify need for unique distributed-system-id

[gzhou] GEODE-2161: avoid unnecessary byte copy of hdos.

[gzhou] This closes #293

[adongre] GEODE-2141: Replace use of List with ConcurrentHashSet for storing

--
[...truncated 574 lines...]
:geode-json:flakyTest UP-TO-DATE
:geode-json:integrationTest UP-TO-DATE
:geode-junit:javadoc
:geode-junit:javadocJar
:geode-junit:sourcesJar
:geode-junit:signArchives SKIPPED
:geode-junit:assemble
:geode-junit:compileTestJava
:geode-junit:processTestResources UP-TO-DATE
:geode-junit:testClasses
:geode-junit:checkMissedTests
:geode-junit:spotlessJavaCheck
:geode-junit:spotlessCheck
:geode-junit:test
:geode-junit:check
:geode-junit:build
:geode-junit:distributedTest
:geode-junit:flakyTest
:geode-junit:integrationTest
:geode-lucene:assemble
:geode-lucene:compileTestJavaNote: Some input files use or override a 
deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

:geode-lucene:processTestResources
:geode-lucene:testClasses
:geode-lucene:checkMissedTests
:geode-lucene:spotlessJavaCheck
:geode-lucene:spotlessCheck
:geode-lucene:test
:geode-lucene:check
:geode-lucene:build
:geode-lucene:distributedTest
:geode-lucene:flakyTest
:geode-lucene:integrationTest
:geode-old-client-support:assemble
:geode-old-client-support:compileTestJava
:geode-old-client-support:processTestResources UP-TO-DATE
:geode-old-client-support:testClasses
:geode-old-client-support:checkMissedTests
:geode-old-client-support:spotlessJavaCheck
:geode-old-client-support:spotlessCheck
:geode-old-client-support:test
:geode-old-client-support:check
:geode-old-client-support:build
:geode-old-client-support:distributedTest
:geode-old-client-support:flakyTest
:geode-old-client-support:integrationTest
:geode-pulse:assemble
:geode-pulse:compileTestJavaNote: 

 uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

:geode-pulse:processTestResources
:geode-pulse:testClasses
:geode-pulse:checkMissedTests
:geode-pulse:spotlessJavaCheck
:geode-pulse:spotlessCheck
:geode-pulse:test
:geode-pulse:check
:geode-pulse:build
:geode-pulse:distributedTest
:geode-pulse:flakyTest
:geode-pulse:integrationTest
:geode-rebalancer:assemble
:geode-rebalancer:compileTestJava
:geode-rebalancer:processTestResources UP-TO-DATE
:geode-rebalancer:testClasses
:geode-rebalancer:checkMissedTests
:geode-rebalancer:spotlessJavaCheck
:geode-rebalancer:spotlessCheck
:geode-rebalancer:test
:geode-rebalancer:check
:geode-rebalancer:build
:geode-rebalancer:distributedTest
:geode-rebalancer:flakyTest
:geode-rebalancer:integrationTest
:geode-wan:assemble
:geode-wan:compileTestJavaNote: Some input files use or override a deprecated 
API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

:geode-wan:processTestResources
:geode-wan:testClasses
:geode-wan:checkMissedTests
:geode-wan:spotlessJavaCheck
:geode-wan:spotlessCheck
:geode-wan:test
:geode-wan:check
:geode-wan:build
:geode-wan:distributedTest
:geode-wan:flakyTest
:geode-wan:integrationTest
:geode-web:assemble
:geode-web:compileTestJavaNote: Some input files use or override a deprecated 
API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

:geode-web:processTestResources UP-TO-DATE
:geode-web:testClasses
:geode-web:checkMissedTests
:geode-web:spotlessJavaCheck
:geode-web:spotlessCheck
:geode-web:test
:geode-web:check
:geode-web:build
:geode-web:distributedTest
:geode-web:flakyTest
:geode-web:integrationTest
:geode-web-api:assemble
:geode-web-api:compileTestJava UP-TO-DATE
:geode-web-api:processTestResources UP-TO-DATE
:geode-web-api:testClasses UP-TO-DATE
:geode-web-api:checkMissedTests UP-TO-DATE
:geode-web-api:spotlessJavaCheck
:geode-web-api:spotlessCheck
:geode-web-api:test UP-TO-DATE
:geode-web-api:check
:geode-web-api:build
:geode-web-api:distributedTest UP-TO-DATE
:geode-web-api:flakyTest UP-TO-DATE
:geode-web-api:integrationTest UP-

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

[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.


Repository: geode


Description
---

GEODE-2146 document new privileges req'd for deploy


Diffs
-

  geode-docs/managing/security/implementing_authorization.html.md.erb 
ec47133cd433be44b189628a44a780dbee49e6ee 

Diff: https://reviews.apache.org/r/54315/diff/


Testing
---

gradle rat check passes

Building the book finds no broken links.


Thanks,

Karen Miller

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

[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.m., Karen Miller wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54315/
> ---
> 
> (Updated Dec. 2, 2016, 5:46 p.m.)
> 
> 
> Review request for geode, Dave Barnes, Jinmei Liao, and Joey McAllister.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-2146 document new privileges req'd for deploy
> 
> 
> Diffs
> -
> 
>   geode-docs/managing/security/implementing_authorization.html.md.erb 
> ec47133cd433be44b189628a44a780dbee49e6ee 
> 
> Diff: https://reviews.apache.org/r/54315/diff/
> 
> 
> Testing
> ---
> 
> gradle rat check passes
> 
> Building the book finds no broken links.
> 
> 
> Thanks,
> 
> Karen Miller
> 
>

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

[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., Karen Miller wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54315/
> ---
> 
> (Updated Dec. 2, 2016, 5:46 p.m.)
> 
> 
> Review request for geode, Dave Barnes, Jinmei Liao, and Joey McAllister.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-2146 document new privileges req'd for deploy
> 
> 
> Diffs
> -
> 
>   geode-docs/managing/security/implementing_authorization.html.md.erb 
> ec47133cd433be44b189628a44a780dbee49e6ee 
> 
> Diff: https://reviews.apache.org/r/54315/diff/
> 
> 
> Testing
> ---
> 
> gradle rat check passes
> 
> Building the book finds no broken links.
> 
> 
> Thanks,
> 
> Karen Miller
> 
>

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

[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.com/kjduling/incubator-geode feature/GEODE-2169

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-geode/pull/302.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #302


commit af988dcf9dddb64f322bbb69364d5bb4cbaf68f9
Author: Kevin J. Duling 
Date:   2016-12-02T18:31:41Z

GEODE-2196: Subclasses of OperationContext should also be deprecated.




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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:38 p.m., Ken Howe wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/53857/
> ---
> 
> (Updated Nov. 17, 2016, 10:38 p.m.)
> 
> 
> Review request for geode, anilkumar gingade, Darrel Schneider, and Eric Shu.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> Changed checks for expected log messages from time based to event based.
> 
> Tests that verify that log messages occured as expected were using an overly 
> optimistic test for 'n'
> logs after a set period of time. The Mockito.verify calls are now placed 
> within the swait()'s. Also simplified tests that have a final check to make 
> sure missing colocated region logging stopped after all child PRs are created.
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentColocatedPartitionedRegionDUnitTest.java
>  a044a612fbbc6b905e3ad1389a9c0d7994f8598a 
> 
> Diff: https://reviews.apache.org/r/53857/diff/
> 
> 
> Testing
> ---
> 
> Ran spotlessCheck (and spotlessApply to fix some formatting)
> 
> Precheckin copmleted successfully
> 
> 
> Thanks,
> 
> Ken Howe
> 
>

Re: TLP Transition Changes Coming

[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 transition items.
> -- Our reporting schedule after the initial three months is February, May,
> August, November.
> -- Once the private mail list has been confirmed as migrated, then all
> non-PMC members will be removed from the list if they have not been already
> - Geode Committer List [2]: It is correct as it was submitted by the
> resolution (as only PMC members were defined), however, yes I do agree that
> even though people were not added to the PMC they should have retained
> committer status.
> - Common Tasks [3]: I already created INFRA-12938 as the Common Task
> ticket. I added those items from your list which I did not have to that
> ticket.
> -- Note: I do not see a Geode Sonar project on https://analysis.apache.org,
> so we will have to request one later.
>
> How would people like to handle re-adding the initial committers? Should it
> be done on an as-needed basis? However, I hope we would not need to vote
> them into committers again for Geode.
>
> Does anyone have any history or experience with this issue?

Committers and PMCs are two completely different things. ASF only
concerns itself
with the PMC composition -- the rest it up to your project to define.

Theoretically you could have a re-qualification for committers, but my
recommendation
will simply be to bulk add all of the existing ones to the LDAP group.

Thanks,
Roman.

Re: TLP Transition Changes Coming

[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 request one later.

Re: TLP Transition Changes Coming

[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 to the LDAP group.

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

[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., Karen Miller wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54315/
> ---
> 
> (Updated Dec. 2, 2016, 5:46 p.m.)
> 
> 
> Review request for geode, Dave Barnes, Jinmei Liao, and Joey McAllister.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-2146 document new privileges req'd for deploy
> 
> 
> Diffs
> -
> 
>   geode-docs/managing/security/implementing_authorization.html.md.erb 
> ec47133cd433be44b189628a44a780dbee49e6ee 
> 
> Diff: https://reviews.apache.org/r/54315/diff/
> 
> 
> Testing
> ---
> 
> gradle rat check passes
> 
> Building the book finds no broken links.
> 
> 
> Thanks,
> 
> Karen Miller
> 
>

Re: TLP Transition Changes Coming

[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 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 to the LDAP group.
>
>

Re: TLP Transition Changes Coming

[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:

> 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 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 to the LDAP group.
> >
> >
>

Re: TLP Transition Changes Coming

[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

[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, Dec 2, 2016 at 12:02 PM, Dan Smith  wrote:

> >
> > 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
>



-- 
~/William

Re: TLP Transition Changes Coming

[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 the answer here is the union of two
lists ;-)

Thanks,
Roman.

Re: TLP Transition Changes Coming

[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 match right before the discussion
> on graduation. At any rate -- I think the answer here is the union of two
> lists ;-)
> 
> Thanks,
> Roman.

Review Request 54321: GEODE-2156 Remove incubating references

[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 
Miller, Mark Bretl, and Dan Smith.


Repository: geode


Description
---

- Remove DISCLAIMER files
- Remove "(incubating)" from project name references
- Remove .incubator from website references

Update mailing list references

Remove incubating references from the website

Update repo references


Diffs
-

  .travis.yml e61cde51b8bee2667a6c2b1ab49c69ea20b440fe 
  BUILDING.md 5e1e7933c59ea66211cf290191ab1f8cef8b2afd 
  DISCLAIMER PRE-CREATION 
  NOTICE 4801b2aedf4f254c85cba9f27d2f99d2ca2e046b 
  README.md 955d45f4f5dd4b95f84dd89d7fcf1108678c5ae8 
  dev-tools/docker/base/Dockerfile 1cce0ddb0d29c9e188d27481c82123e357c5b685 
  dev-tools/docker/compile/Dockerfile 6ae343a70eccf7cc4b303228118cee0b8579e79a 
  dev-tools/docker/docs/Dockerfile 8ce7397efb584ceb3ef032d26f13795e328971fc 
  docker/Dockerfile a42b47a3328dfff981786ce113f927888e45763a 
  geode-assembly/src/main/dist/DISCLAIMER PRE-CREATION 
  geode-assembly/src/main/dist/NOTICE 83d0f3313a01881c9c651077036c17f962cd38b9 
  geode-book/README.md d69354bce6fa684d635b1c03643638c7dee33251 
  geode-book/config.yml bbc8ccc2bdffdc5a16134993ebd0683ba71e6d11 
  geode-book/redirects.rb 81f27c834f78a5b188bb5556bfc93645fbe73262 
  
geode-core/src/main/java/org/apache/geode/cache/partition/PartitionListenerAdapter.java
 b56cdf3547811cbae0b3716b83c4d2fe56b51541 
  geode-core/src/main/java/org/apache/geode/internal/i18n/LocalizedStrings.java 
59231232bb85df22e3ca30b1bec338ef0ae033cb 
  geode-docs/about_geode.html.md.erb 0d7f9b8d9ebe80d8b6a904e584fb76ba9fa6e66b 
  geode-docs/getting_started/installation/install_standalone.html.md.erb 
9a1987e0de5b5f821d0dfc55a113e39fe80f65db 
  geode-examples/README.md 62666035ab1bde5f9d5548e887e0133f006c2ef9 
  geode-pulse/src/main/webapp/DataBrowser.html 
a01df6b280decf8acd96ff6ad2f7a494a53ae682 
  geode-pulse/src/main/webapp/Login.html 
f093e38082a1b58a81ba936582599d28e3d2e585 
  geode-pulse/src/main/webapp/META-INF/NOTICE 
706eda265cf0b54c73d01df85c0a79bde9082d8a 
  geode-pulse/src/main/webapp/MemberDetails.html 
bc946da8b175d23d9eba529663e1bb86c036079b 
  geode-pulse/src/main/webapp/QueryStatistics.html 
543e003a0b0c447e3fe8d8e6e44d0329a168b029 
  geode-pulse/src/main/webapp/clusterDetail.html 
b1fc1e0a24821e40bd08a62ad53b0324102ac03c 
  geode-pulse/src/main/webapp/regionDetail.html 
711ef31bd6f22636420cccd20308b4f7ac7cc769 
  geode-site/website/README.md 56b6aeb4ab34359ffd771ce21609c3f403397dd5 
  geode-site/website/content/community/index.html 
34e085c77e8803bfe7e6ac621b2b149f8c46b653 
  geode-site/website/content/docs/index.html 
0a475f95b3425624b8478ca4ce55c946140b4cb3 
  geode-site/website/content/index.html 
55311d77aafa7e9065e27130b8662987b90e2563 
  geode-site/website/content/releases/index.html 
de0335d6bc0edd299edf44b28cdddf7f33fa15c1 
  geode-site/website/layouts/footer.html 
4877109cbd3599dff81d4d5d69a6ab1c9c73d982 
  geode-site/website/layouts/header.html 
618d33d43506ed6a75326998193af224c2ba6a05 
  geode-spark-connector/doc/2_quick.md 70e1a170dcb0f0c7699c7431a1061e3ab75c2635 
  geode-web-api/src/main/webapp/META-INF/NOTICE 
21082b6de4fbd7ba4ec4bcb61e9e1bfda5ac162a 
  geode-web/src/main/webapp/META-INF/NOTICE 
8c6e1d7faf3188599305f44c44318f6e6cfd00e6 
  gradle.properties 164d482161b59534fd91bc14c7773140fbec279e 
  gradle/publish.gradle af5ee13a2eb6f6532a0e7c73bfb2591dfde60b30 
  gradle/sonar.gradle 32f18834b054436d29f1b4ca52c145c35b808c60 

Diff: https://reviews.apache.org/r/54321/diff/


Testing
---

I'm holding this until the infrastructure changes are ready.


Thanks,

Anthony Baker

Re: TLP Transition Changes Coming

[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 (under "Community").
> >
> > I believe I updated it to be a pretty close match right before the
> discussion
> > on graduation. At any rate -- I think the answer here is the union of two
> > lists ;-)
> >
> > Thanks,
> > Roman.
>
>

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

[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 website header.


geode-spark-connector/doc/2_quick.md (line 12)


While we are at changing links, I think this be changed to the new docs URL.


- Mark Bretl


On Dec. 2, 2016, 12:25 p.m., Anthony Baker wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54321/
> ---
> 
> (Updated Dec. 2, 2016, 12:25 p.m.)
> 
> 
> Review request for geode, Dave Barnes, Dick Cavender, Joey McAllister, Karen 
> Miller, Mark Bretl, and Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> - Remove DISCLAIMER files
> - Remove "(incubating)" from project name references
> - Remove .incubator from website references
> 
> Update mailing list references
> 
> Remove incubating references from the website
> 
> Update repo references
> 
> 
> Diffs
> -
> 
>   .travis.yml e61cde51b8bee2667a6c2b1ab49c69ea20b440fe 
>   BUILDING.md 5e1e7933c59ea66211cf290191ab1f8cef8b2afd 
>   DISCLAIMER PRE-CREATION 
>   NOTICE 4801b2aedf4f254c85cba9f27d2f99d2ca2e046b 
>   README.md 955d45f4f5dd4b95f84dd89d7fcf1108678c5ae8 
>   dev-tools/docker/base/Dockerfile 1cce0ddb0d29c9e188d27481c82123e357c5b685 
>   dev-tools/docker/compile/Dockerfile 
> 6ae343a70eccf7cc4b303228118cee0b8579e79a 
>   dev-tools/docker/docs/Dockerfile 8ce7397efb584ceb3ef032d26f13795e328971fc 
>   docker/Dockerfile a42b47a3328dfff981786ce113f927888e45763a 
>   geode-assembly/src/main/dist/DISCLAIMER PRE-CREATION 
>   geode-assembly/src/main/dist/NOTICE 
> 83d0f3313a01881c9c651077036c17f962cd38b9 
>   geode-book/README.md d69354bce6fa684d635b1c03643638c7dee33251 
>   geode-book/config.yml bbc8ccc2bdffdc5a16134993ebd0683ba71e6d11 
>   geode-book/redirects.rb 81f27c834f78a5b188bb5556bfc93645fbe73262 
>   
> geode-core/src/main/java/org/apache/geode/cache/partition/PartitionListenerAdapter.java
>  b56cdf3547811cbae0b3716b83c4d2fe56b51541 
>   
> geode-core/src/main/java/org/apache/geode/internal/i18n/LocalizedStrings.java 
> 59231232bb85df22e3ca30b1bec338ef0ae033cb 
>   geode-docs/about_geode.html.md.erb 0d7f9b8d9ebe80d8b6a904e584fb76ba9fa6e66b 
>   geode-docs/getting_started/installation/install_standalone.html.md.erb 
> 9a1987e0de5b5f821d0dfc55a113e39fe80f65db 
>   geode-examples/README.md 62666035ab1bde5f9d5548e887e0133f006c2ef9 
>   geode-pulse/src/main/webapp/DataBrowser.html 
> a01df6b280decf8acd96ff6ad2f7a494a53ae682 
>   geode-pulse/src/main/webapp/Login.html 
> f093e38082a1b58a81ba936582599d28e3d2e585 
>   geode-pulse/src/main/webapp/META-INF/NOTICE 
> 706eda265cf0b54c73d01df85c0a79bde9082d8a 
>   geode-pulse/src/main/webapp/MemberDetails.html 
> bc946da8b175d23d9eba529663e1bb86c036079b 
>   geode-pulse/src/main/webapp/QueryStatistics.html 
> 543e003a0b0c447e3fe8d8e6e44d0329a168b029 
>   geode-pulse/src/main/webapp/clusterDetail.html 
> b1fc1e0a24821e40bd08a62ad53b0324102ac03c 
>   geode-pulse/src/main/webapp/regionDetail.html 
> 711ef31bd6f22636420cccd20308b4f7ac7cc769 
>   geode-site/website/README.md 56b6aeb4ab34359ffd771ce21609c3f403397dd5 
>   geode-site/website/content/community/index.html 
> 34e085c77e8803bfe7e6ac621b2b149f8c46b653 
>   geode-site/website/content/docs/index.html 
> 0a475f95b3425624b8478ca4ce55c946140b4cb3 
>   geode-site/website/content/index.html 
> 55311d77aafa7e9065e27130b8662987b90e2563 
>   geode-site/website/content/releases/index.html 
> de0335d6bc0edd299edf44b28cdddf7f33fa15c1 
>   geode-site/website/layouts/footer.html 
> 4877109cbd3599dff81d4d5d69a6ab1c9c73d982 
>   geode-site/website/layouts/header.html 
> 618d33d43506ed6a75326998193af224c2ba6a05 
>   geode-spark-connector/doc/2_quick.md 
> 70e1a170dcb0f0c7699c7431a1061e3ab75c2635 
>   geode-web-api/src/main/webapp/META-INF/NOTICE 
> 21082b6de4fbd7ba4ec4bcb61e9e1bfda5ac162a 
>   geode-web/src/main/webapp/META-INF/NOTICE 
> 8c6e1d7faf3188599305f44c44318f6e6cfd00e6 
>   gradle.properties 164d482161b59534fd91bc14c7773140fbec279e 
>   gradle/publish.gradle af5ee13a2eb6f6532a0e7c73bfb2591dfde60b30 
>   gradle/sonar.gradle 32f18834b054436d29f1b4ca52c145c35b808c60 
> 
> Diff: https://reviews.apache.org/r/54321/diff/
> 
> 
> Testing
> ---
> 
> I'm holding this until the infrastructure changes are ready.
> 
> 
> Thanks,
> 
> Anthony Baker
> 
>

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

[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 with the previous fix for this issue.  toData and
fromData methods in InternalDistributedMember also need to look at the
version of the data stream in order to decide whether to send the UUID
bytes or not.

View the changeset: 
https://github.com/apache/incubator-geode/compare/612a438720a7...a46cc976efa5

View the full build log and details: 
https://travis-ci.org/apache/incubator-geode/builds/180777848

--

You can configure recipients for build notifications in your .travis.yml file. 
See https://docs.travis-ci.com/user/notifications

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

[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
---

Removing from the map when a cq is closed
Did not protect this map because "misses" or missing entries will just be added 
next lookup.


Diffs
-

  
geode-cq/src/main/java/org/apache/geode/cache/query/internal/cq/CqServiceImpl.java
 1414e90 
  
geode-cq/src/test/java/org/apache/geode/cache/query/internal/cq/CqServiceUnitTest.java
 PRE-CREATION 

Diff: https://reviews.apache.org/r/54323/diff/


Testing
---


Thanks,

Jason Huynh

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

[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 p.m., Ken Howe wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/53857/
> ---
> 
> (Updated Nov. 17, 2016, 2:38 p.m.)
> 
> 
> Review request for geode, anilkumar gingade, Darrel Schneider, and Eric Shu.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> Changed checks for expected log messages from time based to event based.
> 
> Tests that verify that log messages occured as expected were using an overly 
> optimistic test for 'n'
> logs after a set period of time. The Mockito.verify calls are now placed 
> within the swait()'s. Also simplified tests that have a final check to make 
> sure missing colocated region logging stopped after all child PRs are created.
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentColocatedPartitionedRegionDUnitTest.java
>  a044a612fbbc6b905e3ad1389a9c0d7994f8598a 
> 
> Diff: https://reviews.apache.org/r/53857/diff/
> 
> 
> Testing
> ---
> 
> Ran spotlessCheck (and spotlessApply to fix some formatting)
> 
> Precheckin copmleted successfully
> 
> 
> Thanks,
> 
> Ken Howe
> 
>

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

[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/cache/query/internal/cq/CqServiceImpl.java
 (line 1058)


I think this could use computeIfAbsent rather than get followed by 
putIfAbsent.


- Dan Smith


On Dec. 2, 2016, 9:47 p.m., Jason Huynh wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54323/
> ---
> 
> (Updated Dec. 2, 2016, 9:47 p.m.)
> 
> 
> Review request for geode and Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> Removing from the map when a cq is closed
> Did not protect this map because "misses" or missing entries will just be 
> added next lookup.
> 
> 
> Diffs
> -
> 
>   
> geode-cq/src/main/java/org/apache/geode/cache/query/internal/cq/CqServiceImpl.java
>  1414e90 
>   
> geode-cq/src/test/java/org/apache/geode/cache/query/internal/cq/CqServiceUnitTest.java
>  PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/54323/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Jason Huynh
> 
>

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

[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
---

Making sure that the mbean returns false for isConnected if the gateway
is not actually connected to the remote side.


Diffs
-

  
geode-wan/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderEventRemoteDispatcher.java
 16b196586e62dcd3707bd9f70406ff8d5d963308 
  
geode-wan/src/test/java/org/apache/geode/management/WANManagementDUnitTest.java 
9a6cc104a1368fa762bffd1efa1978f007819604 

Diff: https://reviews.apache.org/r/54330/diff/


Testing
---


Thanks,

Dan Smith

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

[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., Dan Smith wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54330/
> ---
> 
> (Updated Dec. 2, 2016, 11:48 p.m.)
> 
> 
> Review request for geode and Barry Oglesby.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> Making sure that the mbean returns false for isConnected if the gateway
> is not actually connected to the remote side.
> 
> 
> Diffs
> -
> 
>   
> geode-wan/src/main/java/org/apache/geode/internal/cache/wan/GatewaySenderEventRemoteDispatcher.java
>  16b196586e62dcd3707bd9f70406ff8d5d963308 
>   
> geode-wan/src/test/java/org/apache/geode/management/WANManagementDUnitTest.java
>  9a6cc104a1368fa762bffd1efa1978f007819604 
> 
> Diff: https://reviews.apache.org/r/54330/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Dan Smith
> 
>

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

[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.


Repository: geode


Description
---

GEODE-2178 - Docs: Update HTTP session management version numbers


Diffs
-

  geode-docs/tools_modules/http_session_mgmt/quick_start.html.md.erb 
cfab15a9e6834ba455afc2e7064f39bb0212a8fa 
  geode-docs/tools_modules/http_session_mgmt/tc_additional_info.html.md.erb 
1e01978891bc3882e06c8e6a4dc7387c471c4ec6 
  
geode-docs/tools_modules/http_session_mgmt/tc_installing_the_module.html.md.erb 
fdc199d90057a5760e12f0702c4a050d0628b9f6 
  
geode-docs/tools_modules/http_session_mgmt/tc_setting_up_the_module.html.md.erb 
aa4244a61552dfa4378b4c638e241064cc6fbee4 
  
geode-docs/tools_modules/http_session_mgmt/tomcat_setting_up_the_module.html.md.erb
 dc805b6fa9e9866dd2d9dbd64e6c155f95c581f9 
  
geode-docs/tools_modules/http_session_mgmt/weblogic_setting_up_the_module.html.md.erb
 e0eb267903eca1c7180ecab884967e41b73a21e9 

Diff: https://reviews.apache.org/r/54332/diff/


Testing
---


Thanks,

Dave Barnes

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

[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 30, 2016.

View the changeset: 
https://github.com/apache/incubator-geode/compare/7be3f99e3965...a65f85d18736

View the full build log and details: 
https://travis-ci.org/apache/incubator-geode/builds/180828501

--

You can configure recipients for build notifications in your .travis.yml file. 
See https://docs.travis-ci.com/user/notifications

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

[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 is successful. I tried spotlessApply but it 
shows no changes.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---