Podling Report Reminder - October 2016

2016-10-03 Thread johndament
Dear podling,

This email was sent by an automated system on behalf of the Apache
Incubator PMC. It is an initial reminder to give you plenty of time to
prepare your quarterly board report.

The board meeting is scheduled for Wed, 19 October 2016, 10:30 am PDT.
The report for your podling will form a part of the Incubator PMC
report. The Incubator PMC requires your report to be submitted 2 weeks
before the board meeting, to allow sufficient time for review and
submission (Wed, October 05).

Please submit your report with sufficient time to allow the Incubator
PMC, and subsequently board members to review and digest. Again, the
very latest you should submit your report is 2 weeks prior to the board
meeting.

Thanks,

The Apache Incubator PMC

Submitting your Report

--

Your report should contain the following:

*   Your project name
*   A brief description of your project, which assumes no knowledge of
the project or necessarily of its field
*   A list of the three most important issues to address in the move
towards graduation.
*   Any issues that the Incubator PMC or ASF Board might wish/need to be
aware of
*   How has the community developed since the last report
*   How has the project developed since the last report.

This should be appended to the Incubator Wiki page at:

http://wiki.apache.org/incubator/October2016

Note: This is manually populated. You may need to wait a little before
this page is created from a template.

Mentors
---

Mentors should review reports for their project(s) and sign them off on
the Incubator wiki page. Signing off reports shows that you are
following the project - projects that are not signed may raise alarms
for the Incubator PMC.

Incubator PMC


Re: Review Request 52488: GEODE-420: fix Pulse test when not using any SSLConfig

2016-10-03 Thread Jared Stewart

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52488/#review151256
---


Ship it!




Ship It!

- Jared Stewart


On Oct. 3, 2016, 11:20 p.m., Jinmei Liao wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52488/
> ---
> 
> (Updated Oct. 3, 2016, 11:20 p.m.)
> 
> 
> Review request for geode, Jared Stewart, Kevin Duling, Kirk Lund, and Udo 
> Kohlmeyer.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-420: fix Pulse test when not using any SSLConfig
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/main/java/org/apache/geode/management/internal/JettyHelper.java
>  089dbacb056da84d6d6ceb36dad6f09b9ce73890 
>   
> geode-pulse/src/test/java/org/apache/geode/tools/pulse/testbed/driver/PulseUITest.java
>  a2365a2dfa3e221fa50664c3f8d96fae2bd93c86 
>   
> geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/PulseAbstractTest.java
>  5024250f8ff3867e0f527d7ef5fbbd501e2d64ee 
> 
> Diff: https://reviews.apache.org/r/52488/diff/
> 
> 
> Testing
> ---
> 
> precheckin passed
> 
> 
> Thanks,
> 
> Jinmei Liao
> 
>



Re: Review Request 52488: GEODE-420: fix Pulse test when not using any SSLConfig

2016-10-03 Thread Udo Kohlmeyer

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52488/#review151255
---


Ship it!




Ship It!

- Udo Kohlmeyer


On Oct. 3, 2016, 11:20 p.m., Jinmei Liao wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52488/
> ---
> 
> (Updated Oct. 3, 2016, 11:20 p.m.)
> 
> 
> Review request for geode, Jared Stewart, Kevin Duling, Kirk Lund, and Udo 
> Kohlmeyer.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-420: fix Pulse test when not using any SSLConfig
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/main/java/org/apache/geode/management/internal/JettyHelper.java
>  089dbacb056da84d6d6ceb36dad6f09b9ce73890 
>   
> geode-pulse/src/test/java/org/apache/geode/tools/pulse/testbed/driver/PulseUITest.java
>  a2365a2dfa3e221fa50664c3f8d96fae2bd93c86 
>   
> geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/PulseAbstractTest.java
>  5024250f8ff3867e0f527d7ef5fbbd501e2d64ee 
> 
> Diff: https://reviews.apache.org/r/52488/diff/
> 
> 
> Testing
> ---
> 
> precheckin passed
> 
> 
> Thanks,
> 
> Jinmei Liao
> 
>



Re: Review Request 52488: GEODE-420: fix Pulse test when not using any SSLConfig

2016-10-03 Thread Jinmei Liao

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52488/
---

(Updated Oct. 3, 2016, 11:20 p.m.)


Review request for geode, Jared Stewart, Kevin Duling, Kirk Lund, and Udo 
Kohlmeyer.


Changes
---

use default SSLConfig instead of passing in null

test passed


Repository: geode


Description
---

GEODE-420: fix Pulse test when not using any SSLConfig


Diffs
-

  
geode-core/src/main/java/org/apache/geode/management/internal/JettyHelper.java 
089dbacb056da84d6d6ceb36dad6f09b9ce73890 
  
geode-pulse/src/test/java/org/apache/geode/tools/pulse/testbed/driver/PulseUITest.java
 a2365a2dfa3e221fa50664c3f8d96fae2bd93c86 
  
geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/PulseAbstractTest.java
 5024250f8ff3867e0f527d7ef5fbbd501e2d64ee 

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


Testing (updated)
---

precheckin passed


Thanks,

Jinmei Liao



Re: Podling Report Reminder - October 2016

2016-10-03 Thread Dan Smith
Thanks Anthony and Swap! Updated draft:


Geode

Geode is a data management platform that provides real-time, consistent
access
to data-intensive applications throughout widely distributed cloud
architectures.

Geode has been incubating since 2015-04-27.

Three most important issues to address in the move towards graduation:

  1. Expanding the community to include contributors and committers outside
of Pivotal.
  2. Establish a steady release cadence.
  3. Discuss graduation on the geode dev list.

Any issues that the Incubator PMC (IPMC) or ASF Board wish/need to be
aware of?

None.

How has the community developed since the last report?

- Karen Miller was added as a new committer and PPMC member
- 3 geode clubhouses on
   - Domain Driven Design & Reactive Programming w/ Apache Geode
   - What's New with Spring Data Geode
   - Events & Continuous Query
- 3 presentations involving geode at SpringOne:
  - Where Does Apache Geode Fit in CQRS Architectures?
  - Spring Data and In-Memory Data Management in Action
  - Design Tradeoffs in Distributed Systems - How Southwest Airlines Uses Geode
- A geode user group meeting in Tokyo


How has the project developed since the last report?
 - We renamed packages from com.gemstone to org.apache to move us closer to
   graduation
 - The geode documentation was donated to the project
 - We had our third public release 1.0.0-incubating.M3
 - We have cut a release branch and getting close to our final 1.0.0
release, using the apache package names

Date of last release:

  2016-08-23

When were the last committers or PMC members elected?

  Karen Miller 2016-07-18

Signed-off-by:

  [ ](geode) Konstantin Boudnik
  [ ](geode) Chip Childers
  [ ](geode) Justin Erenkrantz
  [ ](geode) Jan Iversen
  [ ](geode) Chris Mattmann
  [ ](geode) William A. Rowe Jr.
  [ ](geode) Roman Shaposhnik


[GitHub] incubator-geode pull request #250: Feature/geode 1948

2016-10-03 Thread kjduling
Github user kjduling commented on a diff in the pull request:

https://github.com/apache/incubator-geode/pull/250#discussion_r81654416
  
--- Diff: 
geode-core/src/main/java/org/apache/geode/distributed/AbstractLauncher.java ---
@@ -156,7 +166,14 @@ protected static Properties 
loadGemFireProperties(final URL url) {
 
 if (url != null) {
   try {
-properties.load(new FileReader(new File(url.toURI(;
+File propertyFile = new File(url.toURI());
--- End diff --

Updated.  testLoadGemFirePropertiesWithNonExistingURL is an existing test 
that exercises this code.


---
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: Podling Report Reminder - October 2016

2016-10-03 Thread Anthony Baker
Third?

> On Oct 3, 2016, at 1:31 PM, Dan Smith  wrote:
> 
> - We had our fourth public release 1.0.0-incubating.M3



signature.asc
Description: Message signed with OpenPGP using GPGMail


[GitHub] incubator-geode issue #247: Feature/geode 1902

2016-10-03 Thread kjduling
Github user kjduling commented on the issue:

https://github.com/apache/incubator-geode/pull/247
  
Updated.


---
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: Podling Report Reminder - October 2016

2016-10-03 Thread Dan Smith
Thanks John! Updated draft below.


Geode

Geode is a data management platform that provides real-time, consistent
access
to data-intensive applications throughout widely distributed cloud
architectures.

Geode has been incubating since 2015-04-27.

Three most important issues to address in the move towards graduation:

  1. Expanding the community to include contributors and committers outside
of
Pivotal.
  2. Establish a steady release cadence.
  3. Discuss graduation on the geode dev list.

Any issues that the Incubator PMC (IPMC) or ASF Board wish/need to be
aware of?

None.

How has the community developed since the last report?

- Karen Miller was added as a new committer and PPMC member
- We held 3 geode clubhouses on
   - Domain Driven Design & Reactive Programming w/ Apache Geode
   - What's New with Spring Data Geode
   - Events & Continuous Query
- 3 presentations involving geode at SpringOne:
  - Where Does Apache Geode Fit in CQRS Architectures?
  - Spring Data and In-Memory Data Management in Action
  - Design Tradeoffs in Distributed Systems - How Southwest Airlines Uses
Geode

How has the project developed since the last report?
 - We renamed packages from com.gemstone to org.apache to move us closer to
   graduation
 - The geode documentation was donated to the project
 - We had our fourth public release 1.0.0-incubating.M3
 - We have cut a release branch and getting close to our final 1.0.0
release,
   using the apache package names

Date of last release:

  2016-08-23

When were the last committers or PMC members elected?

  Karen Miller 2016-07-18

Signed-off-by:

  [ ](geode) Konstantin Boudnik
  [ ](geode) Chip Childers
  [ ](geode) Justin Erenkrantz
  [ ](geode) Jan Iversen
  [ ](geode) Chris Mattmann
  [ ](geode) William A. Rowe Jr.
  [ ](geode) Roman Shaposhnik

Shepherd/Mentor notes:


Re: Review Request 52488: GEODE-420: fix Pulse test when not using any SSLConfig

2016-10-03 Thread Jinmei Liao


> On Oct. 3, 2016, 6:54 p.m., Udo Kohlmeyer wrote:
> > geode-core/src/main/java/org/apache/geode/management/internal/JettyHelper.java,
> >  line 200
> > 
> >
> > I don't think you should pass null. The SSLConfigurationFactory will 
> > always provide you with a SSLConfig object. This object might be a 
> > ssl-enabled=false, but that is ok. I think this should should be replaced 
> > with:
> > 
> > SSLConfigurationFactory.setDistributionConfig(new DistributionConfig);
> > final Server jetty = JettyHelper.initJetty(null, 8090, 
> > SSLConfigurationFactory.getSSLConfigForComponent(SecurableCommunicationChannel.WEB));

DistributionConfig is an interface, and if I use DistributionConfigImpl(), I 
get compiling error as well. I understand your concern of "checking null", but 
by forcing callers to pass in a valid SSLConfig while there is no easy way for 
them to get a default-not-enabled one gives the caller unnecessary burden. We 
can either allow null here, or have SSLConfigurationFactory add a method to 
return a default-nothing-enabled config?


- Jinmei


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52488/#review151207
---


On Oct. 3, 2016, 5:52 p.m., Jinmei Liao wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52488/
> ---
> 
> (Updated Oct. 3, 2016, 5:52 p.m.)
> 
> 
> Review request for geode, Jared Stewart, Kevin Duling, Kirk Lund, and Udo 
> Kohlmeyer.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-420: fix Pulse test when not using any SSLConfig
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/main/java/org/apache/geode/management/internal/JettyHelper.java
>  089dbacb056da84d6d6ceb36dad6f09b9ce73890 
>   
> geode-pulse/src/test/java/org/apache/geode/tools/pulse/testbed/driver/PulseUITest.java
>  a2365a2dfa3e221fa50664c3f8d96fae2bd93c86 
>   
> geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/PulseAbstractTest.java
>  5024250f8ff3867e0f527d7ef5fbbd501e2d64ee 
> 
> Diff: https://reviews.apache.org/r/52488/diff/
> 
> 
> Testing
> ---
> 
> precheckin running
> 
> 
> Thanks,
> 
> Jinmei Liao
> 
>



Re: Podling Report Reminder - October 2016

2016-10-03 Thread John Blum
Dan-

Luke and I also presented at *SpringOne*, talking about Apache Geode; see
here...

https://2016.event.springoneplatform.io/schedule/sessions/spring_data_and_in_memory_data_management_in_action.html

*Brain Dunlap* from *Southwest Airlines* also talked (
https://2016.event.springoneplatform.io/schedule/sessions/design_tradeoffs_in_distributed_systems_how_southwest_airlines_uses_geode.html)
about Apache Geode.

Thanks!
-John



On Mon, Oct 3, 2016 at 11:17 AM, Dan Smith  wrote:

> Here's a draft of our podling report for this quarter. Please review and
> give your feedback.
>
> Thanks!
> -Dan
>
> 
> Geode
>
> Geode is a data management platform that provides real-time, consistent
> access
> to data-intensive applications throughout widely distributed cloud
> architectures.
>
> Geode has been incubating since 2015-04-27.
>
> Three most important issues to address in the move towards graduation:
>
>   1. Expanding the community to include contributors and committers outside
> of
> Pivotal.
>   2. Establish a steady release cadence.
>   3. Discuss graduation on the geode dev list.
>
> Any issues that the Incubator PMC (IPMC) or ASF Board wish/need to be
> aware of?
>
> None.
>
> How has the community developed since the last report?
>
> - Karen Miller was added as a new committer and PPMC member
> - We held 3 geode clubhouses on
>- Domain Driven Design & Reactive Programming w/ Apache Geode
>- What's New with Spring Data Geode
>- Events & Continuous Query
> - Presentation on Geode at SpringOne: "Where Does Apache Geode Fit in CQRS
>   Architectures?"
>
> How has the project developed since the last report?
>  - We renamed packages from com.gemstone to org.apache to move us closer to
>graduation
>  - The geode documentation was donated to the project
>  - We had our fourth public release 1.0.0-incubating.M3
>  - We have cut a release branch and getting close to our final 1.0.0
> release,
>using the apache package names
>
> Date of last release:
>
>   2016-08-23
>
> When were the last committers or PMC members elected?
>
>   Karen Miller 2016-07-18
>
> Signed-off-by:
>
>   [ ](geode) Konstantin Boudnik
>   [ ](geode) Chip Childers
>   [ ](geode) Justin Erenkrantz
>   [ ](geode) Jan Iversen
>   [ ](geode) Chris Mattmann
>   [ ](geode) William A. Rowe Jr.
>   [ ](geode) Roman Shaposhnik
>
> Shepherd/Mentor notes:
>
>
> On Mon, Oct 3, 2016 at 9:39 AM, Dan Smith  wrote:
>
> > I can do this for this quarter.
> >
> > -Dan
> >
> > On Sun, Oct 2, 2016 at 6:13 AM,  wrote:
> >
> >> Dear podling,
> >>
> >> This email was sent by an automated system on behalf of the Apache
> >> Incubator PMC. It is an initial reminder to give you plenty of time to
> >> prepare your quarterly board report.
> >>
> >> The board meeting is scheduled for Wed, 19 October 2016, 10:30 am PDT.
> >> The report for your podling will form a part of the Incubator PMC
> >> report. The Incubator PMC requires your report to be submitted 2 weeks
> >> before the board meeting, to allow sufficient time for review and
> >> submission (Wed, October 05).
> >>
> >> Please submit your report with sufficient time to allow the Incubator
> >> PMC, and subsequently board members to review and digest. Again, the
> >> very latest you should submit your report is 2 weeks prior to the board
> >> meeting.
> >>
> >> Thanks,
> >>
> >> The Apache Incubator PMC
> >>
> >> Submitting your Report
> >>
> >> --
> >>
> >> Your report should contain the following:
> >>
> >> *   Your project name
> >> *   A brief description of your project, which assumes no knowledge of
> >> the project or necessarily of its field
> >> *   A list of the three most important issues to address in the move
> >> towards graduation.
> >> *   Any issues that the Incubator PMC or ASF Board might wish/need to be
> >> aware of
> >> *   How has the community developed since the last report
> >> *   How has the project developed since the last report.
> >>
> >> This should be appended to the Incubator Wiki page at:
> >>
> >> http://wiki.apache.org/incubator/October2016
> >>
> >> Note: This is manually populated. You may need to wait a little before
> >> this page is created from a template.
> >>
> >> Mentors
> >> ---
> >>
> >> Mentors should review reports for their project(s) and sign them off on
> >> the Incubator wiki page. Signing off reports shows that you are
> >> following the project - projects that are not signed may raise alarms
> >> for the Incubator PMC.
> >>
> >> Incubator PMC
> >>
> >
> >
>



-- 
-John
503-504-8657
john.blum10101 (skype)


Re: Review Request 52488: GEODE-420: fix Pulse test when not using any SSLConfig

2016-10-03 Thread Udo Kohlmeyer

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52488/#review151207
---




geode-core/src/main/java/org/apache/geode/management/internal/JettyHelper.java 
(line 72)


Eventhough it might be "good practice" to check for null,I don't think we 
should check for null. There will always be a SSLConfig object and we should 
expect one. We should rather fix the issue if the SSLConfig is not correctly 
configured.



geode-core/src/main/java/org/apache/geode/management/internal/JettyHelper.java 
(line 198)


I don't think you should pass null. The SSLConfigurationFactory will always 
provide you with a SSLConfig object. This object might be a ssl-enabled=false, 
but that is ok. I think this should should be replaced with:

SSLConfigurationFactory.setDistributionConfig(new DistributionConfig);
final Server jetty = JettyHelper.initJetty(null, 8090, 
SSLConfigurationFactory.getSSLConfigForComponent(SecurableCommunicationChannel.WEB));


- Udo Kohlmeyer


On Oct. 3, 2016, 5:52 p.m., Jinmei Liao wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52488/
> ---
> 
> (Updated Oct. 3, 2016, 5:52 p.m.)
> 
> 
> Review request for geode, Jared Stewart, Kevin Duling, Kirk Lund, and Udo 
> Kohlmeyer.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-420: fix Pulse test when not using any SSLConfig
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/main/java/org/apache/geode/management/internal/JettyHelper.java
>  089dbacb056da84d6d6ceb36dad6f09b9ce73890 
>   
> geode-pulse/src/test/java/org/apache/geode/tools/pulse/testbed/driver/PulseUITest.java
>  a2365a2dfa3e221fa50664c3f8d96fae2bd93c86 
>   
> geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/PulseAbstractTest.java
>  5024250f8ff3867e0f527d7ef5fbbd501e2d64ee 
> 
> Diff: https://reviews.apache.org/r/52488/diff/
> 
> 
> Testing
> ---
> 
> precheckin running
> 
> 
> Thanks,
> 
> Jinmei Liao
> 
>



Re: Geode docs: location of donated source files

2016-10-03 Thread Dave Barnes
That sounds like the end result I was hoping for.

On Mon, Oct 3, 2016 at 10:05 AM, Dan Smith  wrote:

> Sorry, yes, I meant a new geode-docs subdirectory.
>
> -Dan
>
> On Sat, Oct 1, 2016 at 5:12 PM, Anthony Baker  wrote:
>
>> Did you mean a new top-level dir (e.g. geode-docs) or a subdir of
>> geode-core?  Since the docs could cover multiple components I’m not sure it
>> makes sense to put them under geode-core.
>>
>> Anthony
>>
>>
>> >
>> > I think we should just go ahead and create a feature branch off of the
>> the
>> > current geode develop and merge these changes in as a geode-core
>> > subdirectory. We could start the work of cleaning up the build
>> > instructions, etc. right away on that feature branch.
>> >
>> > -Dan
>> >
>>
>>
>


Re: Podling Report Reminder - October 2016

2016-10-03 Thread Dan Smith
Here's a draft of our podling report for this quarter. Please review and
give your feedback.

Thanks!
-Dan


Geode

Geode is a data management platform that provides real-time, consistent
access
to data-intensive applications throughout widely distributed cloud
architectures.

Geode has been incubating since 2015-04-27.

Three most important issues to address in the move towards graduation:

  1. Expanding the community to include contributors and committers outside
of
Pivotal.
  2. Establish a steady release cadence.
  3. Discuss graduation on the geode dev list.

Any issues that the Incubator PMC (IPMC) or ASF Board wish/need to be
aware of?

None.

How has the community developed since the last report?

- Karen Miller was added as a new committer and PPMC member
- We held 3 geode clubhouses on
   - Domain Driven Design & Reactive Programming w/ Apache Geode
   - What's New with Spring Data Geode
   - Events & Continuous Query
- Presentation on Geode at SpringOne: "Where Does Apache Geode Fit in CQRS
  Architectures?"

How has the project developed since the last report?
 - We renamed packages from com.gemstone to org.apache to move us closer to
   graduation
 - The geode documentation was donated to the project
 - We had our fourth public release 1.0.0-incubating.M3
 - We have cut a release branch and getting close to our final 1.0.0
release,
   using the apache package names

Date of last release:

  2016-08-23

When were the last committers or PMC members elected?

  Karen Miller 2016-07-18

Signed-off-by:

  [ ](geode) Konstantin Boudnik
  [ ](geode) Chip Childers
  [ ](geode) Justin Erenkrantz
  [ ](geode) Jan Iversen
  [ ](geode) Chris Mattmann
  [ ](geode) William A. Rowe Jr.
  [ ](geode) Roman Shaposhnik

Shepherd/Mentor notes:


On Mon, Oct 3, 2016 at 9:39 AM, Dan Smith  wrote:

> I can do this for this quarter.
>
> -Dan
>
> On Sun, Oct 2, 2016 at 6:13 AM,  wrote:
>
>> Dear podling,
>>
>> This email was sent by an automated system on behalf of the Apache
>> Incubator PMC. It is an initial reminder to give you plenty of time to
>> prepare your quarterly board report.
>>
>> The board meeting is scheduled for Wed, 19 October 2016, 10:30 am PDT.
>> The report for your podling will form a part of the Incubator PMC
>> report. The Incubator PMC requires your report to be submitted 2 weeks
>> before the board meeting, to allow sufficient time for review and
>> submission (Wed, October 05).
>>
>> Please submit your report with sufficient time to allow the Incubator
>> PMC, and subsequently board members to review and digest. Again, the
>> very latest you should submit your report is 2 weeks prior to the board
>> meeting.
>>
>> Thanks,
>>
>> The Apache Incubator PMC
>>
>> Submitting your Report
>>
>> --
>>
>> Your report should contain the following:
>>
>> *   Your project name
>> *   A brief description of your project, which assumes no knowledge of
>> the project or necessarily of its field
>> *   A list of the three most important issues to address in the move
>> towards graduation.
>> *   Any issues that the Incubator PMC or ASF Board might wish/need to be
>> aware of
>> *   How has the community developed since the last report
>> *   How has the project developed since the last report.
>>
>> This should be appended to the Incubator Wiki page at:
>>
>> http://wiki.apache.org/incubator/October2016
>>
>> Note: This is manually populated. You may need to wait a little before
>> this page is created from a template.
>>
>> Mentors
>> ---
>>
>> Mentors should review reports for their project(s) and sign them off on
>> the Incubator wiki page. Signing off reports shows that you are
>> following the project - projects that are not signed may raise alarms
>> for the Incubator PMC.
>>
>> Incubator PMC
>>
>
>


[GitHub] incubator-geode pull request #251: Feature/geode 1570

2016-10-03 Thread kjduling
GitHub user kjduling opened a pull request:

https://github.com/apache/incubator-geode/pull/251

Feature/geode 1570

precheckin running

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/kjduling/incubator-geode feature/GEODE-1570

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

https://github.com/apache/incubator-geode/pull/251.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 #251


commit 5107660527bfc7322f33fbad97670e607b7e35a8
Author: Kevin Duling 
Date:   2016-09-21T15:50:46Z

GEODE-1570 - developer REST API should be secured

Merged with develop after org.apache package rename

commit b1e252efb71480694c3e235aff2f84968005a59a
Author: Kevin Duling 
Date:   2016-09-21T17:41:06Z

GEODE-1570 - developer REST API should be secured

Moved classes to internal.

commit a7a53e170209e46aa2a0759a41682d091d0fa5ac
Author: Kevin Duling 
Date:   2016-10-03T17:19:57Z

Merge branch 'origin/release/1.0.0-incubating' into feature/GEODE-1570




---
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 52488: GEODE-420: fix Pulse test when not using any SSLConfig

2016-10-03 Thread Jared Stewart

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52488/#review151196
---


Ship it!




Ship It!

- Jared Stewart


On Oct. 3, 2016, 5:52 p.m., Jinmei Liao wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52488/
> ---
> 
> (Updated Oct. 3, 2016, 5:52 p.m.)
> 
> 
> Review request for geode, Jared Stewart, Kevin Duling, Kirk Lund, and Udo 
> Kohlmeyer.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-420: fix Pulse test when not using any SSLConfig
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/main/java/org/apache/geode/management/internal/JettyHelper.java
>  089dbacb056da84d6d6ceb36dad6f09b9ce73890 
>   
> geode-pulse/src/test/java/org/apache/geode/tools/pulse/testbed/driver/PulseUITest.java
>  a2365a2dfa3e221fa50664c3f8d96fae2bd93c86 
>   
> geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/PulseAbstractTest.java
>  5024250f8ff3867e0f527d7ef5fbbd501e2d64ee 
> 
> Diff: https://reviews.apache.org/r/52488/diff/
> 
> 
> Testing
> ---
> 
> precheckin running
> 
> 
> Thanks,
> 
> Jinmei Liao
> 
>



Review Request 52488: GEODE-420: fix Pulse test when not using any SSLConfig

2016-10-03 Thread Jinmei Liao

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52488/
---

Review request for geode, Jared Stewart, Kevin Duling, Kirk Lund, and Udo 
Kohlmeyer.


Repository: geode


Description
---

GEODE-420: fix Pulse test when not using any SSLConfig


Diffs
-

  
geode-core/src/main/java/org/apache/geode/management/internal/JettyHelper.java 
089dbacb056da84d6d6ceb36dad6f09b9ce73890 
  
geode-pulse/src/test/java/org/apache/geode/tools/pulse/testbed/driver/PulseUITest.java
 a2365a2dfa3e221fa50664c3f8d96fae2bd93c86 
  
geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/PulseAbstractTest.java
 5024250f8ff3867e0f527d7ef5fbbd501e2d64ee 

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


Testing
---

precheckin running


Thanks,

Jinmei Liao



Re: Review Request 52468: Add FlakyTest category to tests with open bugs

2016-10-03 Thread Anthony Baker


> On Oct. 3, 2016, 5:30 p.m., Dan Smith wrote:
> > I noticed you added additional ticket numbers to the same tests in a couple 
> > of places. Do we really need multiple tickets for the same test, or would 
> > it make sense to just close one of the tickets as a duplicate?

Usually what's happening is the test was defined in a superclass, but the 
failure occurred when the subclass was run.


- Anthony


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52468/#review151189
---


On Oct. 2, 2016, 12:07 a.m., Anthony Baker wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52468/
> ---
> 
> (Updated Oct. 2, 2016, 12:07 a.m.)
> 
> 
> Review request for geode.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> Annotate test methods with FlakyTest category if there is an open
> bug for that test. This will improve the signal/noise ratio for
> unit/integration/distributed tests.  Flaky tests are still run
> as part of precheckin and flakyTest targets.
> 
> 
> Diffs
> -
> 
>   
> extensions/geode-modules-session/src/test/java/org/apache/geode/modules/session/internal/filter/SessionReplicationIntegrationJUnitTest.java
>  PRE-CREATION 
>   
> geode-assembly/src/test/java/org/apache/geode/management/internal/configuration/SharedConfigurationEndToEndDUnitTest.java
>  PRE-CREATION 
>   
> geode-assembly/src/test/java/org/apache/geode/rest/internal/web/controllers/RestAPIsOnGroupsFunctionExecutionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/ConnectionPoolDUnitTest.java 
> PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/management/MemoryThresholdsOffHeapDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/dunit/CompiledInDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/dunit/QueryUsingFunctionContextDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/internal/index/ConcurrentIndexOperationsOnOverflowRegionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/partitioned/PRQueryCacheCloseDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/partitioned/PRQueryRegionCloseDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/DistributedSystemDUnitTest.java
>  PRE-CREATION 
>   geode-core/src/test/java/org/apache/geode/distributed/LocatorDUnitTest.java 
> PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/LocatorLauncherTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/ServerLauncherTest.java 
> PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/SystemAdminDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/internal/ConsoleDistributionManagerDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/internal/membership/MembershipJUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/FixedPRSinglehopDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/PartitionedRegionSingleHopDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/ClientServerFunctionExecutionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/FunctionServiceBase.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRClientServerRegionFunctionExecutionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRClientServerRegionFunctionExecutionFailoverDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRClientServerRegionFunctionExecutionSelectorNoSingleHopDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRColocationDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentPartitionedRegionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/fixed/FixedPartitioningDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/Bug36805DUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/DurableRegistrationDUnitTest.java
>  PRE-CREATION 
>   
> 

Re: Review Request 52468: Add FlakyTest category to tests with open bugs

2016-10-03 Thread Dan Smith

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52468/#review151189
---


Ship it!




I noticed you added additional ticket numbers to the same tests in a couple of 
places. Do we really need multiple tickets for the same test, or would it make 
sense to just close one of the tickets as a duplicate?

- Dan Smith


On Oct. 2, 2016, 12:07 a.m., Anthony Baker wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52468/
> ---
> 
> (Updated Oct. 2, 2016, 12:07 a.m.)
> 
> 
> Review request for geode.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> Annotate test methods with FlakyTest category if there is an open
> bug for that test. This will improve the signal/noise ratio for
> unit/integration/distributed tests.  Flaky tests are still run
> as part of precheckin and flakyTest targets.
> 
> 
> Diffs
> -
> 
>   
> extensions/geode-modules-session/src/test/java/org/apache/geode/modules/session/internal/filter/SessionReplicationIntegrationJUnitTest.java
>  PRE-CREATION 
>   
> geode-assembly/src/test/java/org/apache/geode/management/internal/configuration/SharedConfigurationEndToEndDUnitTest.java
>  PRE-CREATION 
>   
> geode-assembly/src/test/java/org/apache/geode/rest/internal/web/controllers/RestAPIsOnGroupsFunctionExecutionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/ConnectionPoolDUnitTest.java 
> PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/management/MemoryThresholdsOffHeapDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/dunit/CompiledInDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/dunit/QueryUsingFunctionContextDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/internal/index/ConcurrentIndexOperationsOnOverflowRegionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/partitioned/PRQueryCacheCloseDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/partitioned/PRQueryRegionCloseDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/DistributedSystemDUnitTest.java
>  PRE-CREATION 
>   geode-core/src/test/java/org/apache/geode/distributed/LocatorDUnitTest.java 
> PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/LocatorLauncherTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/ServerLauncherTest.java 
> PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/SystemAdminDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/internal/ConsoleDistributionManagerDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/internal/membership/MembershipJUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/FixedPRSinglehopDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/PartitionedRegionSingleHopDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/ClientServerFunctionExecutionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/FunctionServiceBase.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRClientServerRegionFunctionExecutionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRClientServerRegionFunctionExecutionFailoverDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRClientServerRegionFunctionExecutionSelectorNoSingleHopDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRColocationDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentPartitionedRegionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/fixed/FixedPartitioningDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/Bug36805DUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/DurableRegistrationDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/wan/asyncqueue/AsyncEventListenerDUnitTest.java
>  PRE-CREATION 
>   
> 

Re: buildSrc prevents geode-core:test

2016-10-03 Thread Kirk Lund
The --tests syntax works well for me. Thanks!


On Fri, Sep 30, 2016 at 4:18 PM, Jared Stewart  wrote:

> I don’t know why that happens, but I believe -Dtest.single has been
> deprecated in favor of --tests which lets you specify a test class pattern
> to match against.  The following works for me:
>
> ./gradlew geode-core:test --tests *QueryDataFunctionApplyLimitClauseTest
>
> (Note that the * is necessary above to avoid having to fully qualify
> org.apache.geode.management.internal.beans.QueryDataFunctionApplyLimitCla
> useTest.)
>
> Best,
> Jared
>
>
> > On Sep 30, 2016, at 4:03 PM, Kirk Lund  wrote:
> >
> > Does anyone know why "buildSrc:test" executes when I try to execute
> > "geode-core:test"? See my output below. Instead of executing
> > "geode-core:test", gradle executes "buildSrc:test" and fails with "Could
> > not find matching test for pattern" -- I think there's something broken
> in
> > our gradle files.
> >
> > Thanks,
> > Kirk
> >
> > /Users/klund/dev/geode [530]$ find . -name
> > QueryDataFunctionApplyLimitClauseTest.java
> > ./geode-core/src/test/java/org/apache/geode/management/internal/beans/
> QueryDataFunctionApplyLimitClauseTest.java
> >
> > /Users/klund/dev/geode [531]$ ./gradlew
> > geode-core:test -Dtest.single=QueryDataFunctionApplyLimitClauseTest
> > :buildSrc:compileJava UP-TO-DATE
> > :buildSrc:compileGroovy UP-TO-DATE
> > :buildSrc:processResources UP-TO-DATE
> > :buildSrc:classes UP-TO-DATE
> > :buildSrc:jar UP-TO-DATE
> > :buildSrc:assemble UP-TO-DATE
> > :buildSrc:compileTestJava UP-TO-DATE
> > :buildSrc:compileTestGroovy UP-TO-DATE
> > :buildSrc:processTestResources UP-TO-DATE
> > :buildSrc:testClasses UP-TO-DATE
> > :buildSrc:test FAILED
> >
> > FAILURE: Build failed with an exception.
> >
> > * What went wrong:
> > Execution failed for task ':test'.
> >> Could not find matching test for pattern:
> > QueryDataFunctionApplyLimitClauseTest
> >
> > * Try:
> > Run with --stacktrace option to get the stack trace. Run with --info or
> > --debug option to get more log output.
> >
> > BUILD FAILED
> >
> > Total time: 0.62 secs
> > /Users/klund/dev/geode [532]$
>
>


Re: Review Request 52468: Add FlakyTest category to tests with open bugs

2016-10-03 Thread Scott Jewell

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52468/#review151186
---


Ship it!




Ship It!

- Scott Jewell


On Oct. 2, 2016, 12:07 a.m., Anthony Baker wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52468/
> ---
> 
> (Updated Oct. 2, 2016, 12:07 a.m.)
> 
> 
> Review request for geode.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> Annotate test methods with FlakyTest category if there is an open
> bug for that test. This will improve the signal/noise ratio for
> unit/integration/distributed tests.  Flaky tests are still run
> as part of precheckin and flakyTest targets.
> 
> 
> Diffs
> -
> 
>   
> extensions/geode-modules-session/src/test/java/org/apache/geode/modules/session/internal/filter/SessionReplicationIntegrationJUnitTest.java
>  PRE-CREATION 
>   
> geode-assembly/src/test/java/org/apache/geode/management/internal/configuration/SharedConfigurationEndToEndDUnitTest.java
>  PRE-CREATION 
>   
> geode-assembly/src/test/java/org/apache/geode/rest/internal/web/controllers/RestAPIsOnGroupsFunctionExecutionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/ConnectionPoolDUnitTest.java 
> PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/management/MemoryThresholdsOffHeapDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/dunit/CompiledInDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/dunit/QueryUsingFunctionContextDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/internal/index/ConcurrentIndexOperationsOnOverflowRegionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/partitioned/PRQueryCacheCloseDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/partitioned/PRQueryRegionCloseDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/DistributedSystemDUnitTest.java
>  PRE-CREATION 
>   geode-core/src/test/java/org/apache/geode/distributed/LocatorDUnitTest.java 
> PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/LocatorLauncherTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/ServerLauncherTest.java 
> PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/SystemAdminDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/internal/ConsoleDistributionManagerDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/internal/membership/MembershipJUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/FixedPRSinglehopDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/PartitionedRegionSingleHopDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/ClientServerFunctionExecutionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/FunctionServiceBase.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRClientServerRegionFunctionExecutionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRClientServerRegionFunctionExecutionFailoverDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRClientServerRegionFunctionExecutionSelectorNoSingleHopDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRColocationDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentPartitionedRegionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/fixed/FixedPartitioningDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/Bug36805DUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/DurableRegistrationDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/wan/asyncqueue/AsyncEventListenerDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/management/CompositeTypeTestDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/management/MemberMBeanAttributesDUnitTest.java
>  PRE-CREATION 
>   
> 

Re: Geode docs: location of donated source files

2016-10-03 Thread Dan Smith
Sorry, yes, I meant a new geode-docs subdirectory.

-Dan

On Sat, Oct 1, 2016 at 5:12 PM, Anthony Baker  wrote:

> Did you mean a new top-level dir (e.g. geode-docs) or a subdir of
> geode-core?  Since the docs could cover multiple components I’m not sure it
> makes sense to put them under geode-core.
>
> Anthony
>
>
> >
> > I think we should just go ahead and create a feature branch off of the
> the
> > current geode develop and merge these changes in as a geode-core
> > subdirectory. We could start the work of cleaning up the build
> > instructions, etc. right away on that feature branch.
> >
> > -Dan
> >
>
>


Re: Podling Report Reminder - October 2016

2016-10-03 Thread Dan Smith
I can do this for this quarter.

-Dan

On Sun, Oct 2, 2016 at 6:13 AM,  wrote:

> Dear podling,
>
> This email was sent by an automated system on behalf of the Apache
> Incubator PMC. It is an initial reminder to give you plenty of time to
> prepare your quarterly board report.
>
> The board meeting is scheduled for Wed, 19 October 2016, 10:30 am PDT.
> The report for your podling will form a part of the Incubator PMC
> report. The Incubator PMC requires your report to be submitted 2 weeks
> before the board meeting, to allow sufficient time for review and
> submission (Wed, October 05).
>
> Please submit your report with sufficient time to allow the Incubator
> PMC, and subsequently board members to review and digest. Again, the
> very latest you should submit your report is 2 weeks prior to the board
> meeting.
>
> Thanks,
>
> The Apache Incubator PMC
>
> Submitting your Report
>
> --
>
> Your report should contain the following:
>
> *   Your project name
> *   A brief description of your project, which assumes no knowledge of
> the project or necessarily of its field
> *   A list of the three most important issues to address in the move
> towards graduation.
> *   Any issues that the Incubator PMC or ASF Board might wish/need to be
> aware of
> *   How has the community developed since the last report
> *   How has the project developed since the last report.
>
> This should be appended to the Incubator Wiki page at:
>
> http://wiki.apache.org/incubator/October2016
>
> Note: This is manually populated. You may need to wait a little before
> this page is created from a template.
>
> Mentors
> ---
>
> Mentors should review reports for their project(s) and sign them off on
> the Incubator wiki page. Signing off reports shows that you are
> following the project - projects that are not signed may raise alarms
> for the Incubator PMC.
>
> Incubator PMC
>


Re: Review Request 52468: Add FlakyTest category to tests with open bugs

2016-10-03 Thread Kevin Duling

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52468/#review151174
---


Ship it!




Ship It!

- Kevin Duling


On Oct. 1, 2016, 5:07 p.m., Anthony Baker wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52468/
> ---
> 
> (Updated Oct. 1, 2016, 5:07 p.m.)
> 
> 
> Review request for geode.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> Annotate test methods with FlakyTest category if there is an open
> bug for that test. This will improve the signal/noise ratio for
> unit/integration/distributed tests.  Flaky tests are still run
> as part of precheckin and flakyTest targets.
> 
> 
> Diffs
> -
> 
>   
> extensions/geode-modules-session/src/test/java/org/apache/geode/modules/session/internal/filter/SessionReplicationIntegrationJUnitTest.java
>  PRE-CREATION 
>   
> geode-assembly/src/test/java/org/apache/geode/management/internal/configuration/SharedConfigurationEndToEndDUnitTest.java
>  PRE-CREATION 
>   
> geode-assembly/src/test/java/org/apache/geode/rest/internal/web/controllers/RestAPIsOnGroupsFunctionExecutionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/ConnectionPoolDUnitTest.java 
> PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/management/MemoryThresholdsOffHeapDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/dunit/CompiledInDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/dunit/QueryUsingFunctionContextDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/internal/index/ConcurrentIndexOperationsOnOverflowRegionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/partitioned/PRQueryCacheCloseDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/partitioned/PRQueryRegionCloseDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/DistributedSystemDUnitTest.java
>  PRE-CREATION 
>   geode-core/src/test/java/org/apache/geode/distributed/LocatorDUnitTest.java 
> PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/LocatorLauncherTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/ServerLauncherTest.java 
> PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/SystemAdminDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/internal/ConsoleDistributionManagerDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/distributed/internal/membership/MembershipJUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/FixedPRSinglehopDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/PartitionedRegionSingleHopDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/ClientServerFunctionExecutionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/FunctionServiceBase.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRClientServerRegionFunctionExecutionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRClientServerRegionFunctionExecutionFailoverDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRClientServerRegionFunctionExecutionSelectorNoSingleHopDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/execute/PRColocationDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentPartitionedRegionDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/fixed/FixedPartitioningDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/Bug36805DUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/tier/sockets/DurableRegistrationDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/internal/cache/wan/asyncqueue/AsyncEventListenerDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/management/CompositeTypeTestDUnitTest.java
>  PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/management/MemberMBeanAttributesDUnitTest.java
>  PRE-CREATION 
>   
>