Jenkins build is back to normal : Geode-nightly #676

[Apache Jenkins Server]

See 

[jira] [Commented] (GEODE-1984) Make GatewaySender destroy a public API

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/GEODE-1984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15727856#comment-15727856
 ] 

ASF GitHub Bot commented on GEODE-1984:
---

Github user davinash commented on the issue:

https://github.com/apache/geode/pull/303
  
Please help to review this PR 


> Make GatewaySender destroy a public API
> ---
>
> Key: GEODE-1984
> URL: https://issues.apache.org/jira/browse/GEODE-1984
> Project: Geode
>  Issue Type: New Feature
>  Components: wan
>Reporter: Barry Oglesby
>Assignee: Avinash Dongre
>
> The internal {{AbstractGatewaySender}} class has a {{destroy}} API to destroy 
> a {{GatewaySender}}. This is currently an internal API. It would be nice to 
> make this public by:
> - adding destroy to the {{GatewaySender}} interface
> - provide {{gfsh}} support



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] geode issue #303: GEODE-1984: Fix Issue Make GatewaySender destroy a public ...

[davinash]

Github user davinash commented on the issue:

https://github.com/apache/geode/pull/303
  
Please help to review this PR 


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

[jira] [Commented] (GEODE-1835) A message logged by the configure pdx command is incorrect

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/GEODE-1835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15727808#comment-15727808
 ] 

ASF GitHub Bot commented on GEODE-1835:
---

Github user ameybarve15 commented on the issue:

https://github.com/apache/geode/pull/304
  
Is this PR ready for commit?


> A message logged by the configure pdx command is incorrect
> --
>
> Key: GEODE-1835
> URL: https://issues.apache.org/jira/browse/GEODE-1835
> Project: Geode
>  Issue Type: Bug
>  Components: gfsh
>Reporter: Barry Oglesby
>Assignee: Amey Barve
>
> The message below is only logged when there are no members. Instead, it 
> should be logged when there are members.
> {noformat}
> The command would only take effect on new data members joining the 
> distributed system. It won't affect the existing data members
> {noformat}
> The condition in {{PDXCommands.configurePDX}} is:
> {noformat}
> if (CliUtil.getAllNormalMembers(CliUtil.getCacheIfExists()).isEmpty()) {
>   ird.addLine(CliStrings.CONFIGURE_PDX__NORMAL__MEMBERS__WARNING);
> }
> {noformat}
> It should test for {{!isEmpty}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] geode issue #304: GEODE-1835: A message logged by the configure pdx command ...

[ameybarve15]

Github user ameybarve15 commented on the issue:

https://github.com/apache/geode/pull/304
  
Is this PR ready for commit?


---
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 54330: GEODE-2179 - Checking for disconnected connection in GatewaySenderMBean

[Amey Barve]

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


Ship it!




Ship It!

- Amey Barve


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

[jira] [Commented] (GEODE-2179) GatewaySenderMBeans isConnected flag does not notice if a connection is destroyed

[Amey Barve (JIRA)]

[ 
https://issues.apache.org/jira/browse/GEODE-2179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15727774#comment-15727774
 ] 

Amey Barve commented on GEODE-2179:
---

Yes now I can view diff and patch too looks good !

> GatewaySenderMBeans isConnected flag does not notice if a connection is 
> destroyed
> -
>
> Key: GEODE-2179
> URL: https://issues.apache.org/jira/browse/GEODE-2179
> Project: Geode
>  Issue Type: Bug
>  Components: wan
>Reporter: Dan Smith
>Assignee: Amey Barve
>
> The GatewaySenderMBean has a method called isConnected which ultimately calls 
> this code in GatewaySenderEventRemoteDispatcher
> {code}
> @Override
> public boolean isConnectedToRemote() {
>   return connection != null;
> }
> {code}
> However, this doesn't take into account that the connection may be destroyed, 
> but not null. Therefore the mbean can incorrectly indicate that the sender is 
> connected.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Re: Top-Level Project Tasks Completed

[Anthony Baker]

I’ve updated a bunch of the wiki content to correctly reference mailing lists, 
repos, etc.  I removed a few references to incubator processes that didn’t seem 
relevant any longer, but these pages could use further polish:

https://cwiki.apache.org/confluence/display/GEODE/Nominating+a+Committer
https://cwiki.apache.org/confluence/display/GEODE/Release+Steps

Anthony

> On Dec 6, 2016, at 3:20 PM, Mark Bretl  wrote:
> 
> Ticket for GitHub link is fixed.
>  https://issues.apache.org/jira/browse/INFRA-13052 
> 
> 
> --Mark
> 
> On Tue, Dec 6, 2016 at 3:04 PM, Mark Bretl  > wrote:
> Ticket for ReviewBoard has now been fixed. 
> https://issues.apache.org/jira/browse/INFRA-13054 
> 
> 
> Ticket for JIRA is being worked on now.
> https://issues.apache.org/jira/browse/INFRA-13055 
> 
> 
> --Mark
> 
> On Tue, Dec 6, 2016 at 2:00 PM, Anthony Baker  > wrote:
> Excellent! Thanks for your help getting through this transition Mark!
> 
> Anthony
> 
> > On Dec 6, 2016, at 1:56 PM, Mark Bretl  > > wrote:
> >
> > Hey Anthony,
> >
> > Thanks for checking these issues. Here are my updates:
> >
> > 1) There is an INFRA ticket, 
> > https://issues.apache.org/jira/browse/INFRA-13052 
> >  
> >  > >
> > 2) I will create a ticket
> > 3) I can create a ticket as well.
> >
> > --Mark
> 
> 
> 

Re: Top-Level Project Tasks Completed

[Mark Bretl]

Ticket for GitHub link is fixed.
 https://issues.apache.org/jira/browse/INFRA-13052

--Mark

On Tue, Dec 6, 2016 at 3:04 PM, Mark Bretl  wrote:

> Ticket for ReviewBoard has now been fixed.
> https://issues.apache.org/jira/browse/INFRA-13054
>
> Ticket for JIRA is being worked on now.
> https://issues.apache.org/jira/browse/INFRA-13055
>
> --Mark
>
> On Tue, Dec 6, 2016 at 2:00 PM, Anthony Baker  wrote:
>
>> Excellent! Thanks for your help getting through this transition Mark!
>>
>> Anthony
>>
>> > On Dec 6, 2016, at 1:56 PM, Mark Bretl  wrote:
>> >
>> > Hey Anthony,
>> >
>> > Thanks for checking these issues. Here are my updates:
>> >
>> > 1) There is an INFRA ticket, https://issues.apache.org/jira
>> /browse/INFRA-13052 
>> > 2) I will create a ticket
>> > 3) I can create a ticket as well.
>> >
>> > --Mark
>>
>>
>

Re: TLP Transition Changes Coming

[Roman Shaposhnik]

On Tue, Dec 6, 2016 at 11:43 AM, Mark Bretl  wrote:
> Roman,
>
> Looks a ticket then. Do you still want to be the 'Lead' for the project? Or
> should we update to PMC Chair since we are a TLP now.

I think PMC Chair makes much more sense.

Thanks,
Roman.

Re: Top-Level Project Tasks Completed

[Mark Bretl]

Ticket for ReviewBoard has now been fixed.
https://issues.apache.org/jira/browse/INFRA-13054

Ticket for JIRA is being worked on now.
https://issues.apache.org/jira/browse/INFRA-13055

--Mark

On Tue, Dec 6, 2016 at 2:00 PM, Anthony Baker  wrote:

> Excellent! Thanks for your help getting through this transition Mark!
>
> Anthony
>
> > On Dec 6, 2016, at 1:56 PM, Mark Bretl  wrote:
> >
> > Hey Anthony,
> >
> > Thanks for checking these issues. Here are my updates:
> >
> > 1) There is an INFRA ticket, https://issues.apache.org/
> jira/browse/INFRA-13052  >
> > 2) I will create a ticket
> > 3) I can create a ticket as well.
> >
> > --Mark
>
>

[GitHub] geode issue #301: UITests actually take screenshots on failure

[jaredjstewart]

Github user jaredjstewart commented on the issue:

https://github.com/apache/geode/pull/301
  
Precheckin passed


---
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] geode issue #296: GEODE-2109 : Calling submit on ExecutionService can cause ...

[upthewaterspout]

Github user upthewaterspout commented on the issue:

https://github.com/apache/geode/pull/296
  
+1 looks good to me.


---
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 54382: GEODE-2164: CqNameToOpSingleEntry NAMES_ARRAY field may be refactored out

[Dan Smith]

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


Ship it!




Ship It!

- Dan Smith


On Dec. 5, 2016, 4:43 p.m., Jason Huynh wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54382/
> ---
> 
> (Updated Dec. 5, 2016, 4:43 p.m.)
> 
> 
> Review request for geode and Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> Changed the names field to be a string[] with a single entry and removed the 
> static field NAMES_ARRAY
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientUpdateMessageImpl.java
>  a202ba2 
> 
> Diff: https://reviews.apache.org/r/54382/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Jason Huynh
> 
>

Re: Top-Level Project Tasks Completed

[Anthony Baker]

Excellent! Thanks for your help getting through this transition Mark!

Anthony

> On Dec 6, 2016, at 1:56 PM, Mark Bretl  wrote:
> 
> Hey Anthony, 
> 
> Thanks for checking these issues. Here are my updates:
> 
> 1) There is an INFRA ticket, 
> https://issues.apache.org/jira/browse/INFRA-13052 
> 
> 2) I will create a ticket
> 3) I can create a ticket as well.
> 
> --Mark

Matching Unix Committer List And Website Committer List

[Mark Bretl]

Note: Moved to Dev list...

Hey Dave,

I am working on adding committers back to the official unix group, which
Whimsy is using. The website will always be a secondary source, however, we
could link to the phonebook instead of listing directly on the website.
Since Whimsy requires credentials, I don't think that would be good choice

I am working on re-adding committers which are missing from the TLP
transition.

--Mark

[1] http://people.apache.org/phonebook.html?unix=geode

On Tue, Dec 6, 2016 at 1:33 PM, Dave Barnes  wrote:

> The whimsy committers roster differs from the Geode website's Community
> list of committers. Is the website list obsolete, or would there be value
> in updating it to match the whimsy list? (Adopting a 2-list policy incurs
> some overhead going forward..)
>

Re: Top-Level Project Tasks Completed

[Anthony Baker]

I just pushed a bunch of source changes, including updates to the website.  
Please review and let me know if you see any problems.

Here’s a few clean up tasks I see:

1) On the github mirror, it says "mirrored from 
git://git.apache.org/incubator-geode.git”.
2) The project “category” for the GEODE JIRA project is “Incubator”.  Mark 
already mentioned the Project Lead and email notifications.
3) The reviewboard notifications are being sent to 
d...@geode.incubator.apache.org.

Anthony


> On Dec 5, 2016, at 5:11 PM, Mark Bretl  wrote:
> 
> Hi Everyone,
> 
> The Apache INFRA team has completed the Top-Level Project for Apache Geode.
> 
> Summary of Changes:
> - Website is https://geode.apache.org/
> - Mailing lists (dev, users, private) now have suffix of '@geode.apache.org'
> - GitHub URL is now https://github.com/apache/geode (It has error of 404
> now, but should resolve in about 7 hours)
> 
> If you have any issues, please email the Dev list at 'dev@geode.apache.org'
> so we can investigate.
> 
> Thanks to everyone for their patience. Now, let's move forward with growing
> this community and completing another release!
> 
> Best Regards,
> 
> Mark Bretl
> V.P. Apache Geode

Re: Travis on github PRs

[Kirk Lund]

Great! Let me know if I can help in anyway.


On Tue, Dec 6, 2016 at 11:54 AM, Anthony Baker  wrote:

> I have some changes queued up to the travis yml on GEODE-2156 (there may
> be something else to be done afterwards).
>
> Anthony
>
>
> > On Dec 6, 2016, at 11:36 AM, Kirk Lund  wrote:
> >
> > Looks like the Travis integration on https://github.com/apache/geode
> needs
> > to be updated. It still refers to incubator-geode and clicking the
> > "Details" link goes to an error page "The repository at
> > apache/incubator-geode was not found."
> >
> > Does anyone who knows how to fix this want to fix it? Or feel free to
> send
> > me info on what to do and I'll set aside time to learn how to do this
> (not
> > sure I have permissions on github though).
> >
> > Thanks,
> > Kirk
>
>

Re: Review Request 54436: GEODE-2183: Region.query behaves differently on client and server when full syntax is passed as a predicate

[Dan Smith]

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


Ship it!




I know one thing Region.query is used for is for querying the local data within 
the context of a function. Maybe we need to find a better way to expose that 
capability through the query service if we are going to deprecate/remove 
capabilities from region.query.

- Dan Smith


On Dec. 6, 2016, 7:01 p.m., Jason Huynh wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54436/
> ---
> 
> (Updated Dec. 6, 2016, 7:01 p.m.)
> 
> 
> Review request for geode, Barry Oglesby, nabarun nag, and Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> Added deconstruction/reconstruction code to server side region.query 
> execution to make it consistent with client side.
> 
> At some point I think we were going to deprecate the ability to execute a 
> full select query string from the region.query() method itself.
> 
> Minor prevention of executing a full query with a region that does not match 
> the current region name and prevents multiple region queries from being 
> executed from region.query.  Long term we should restrict this even more to 
> prevent security issues.
> 
> 
> Diffs
> -
> 
>   geode-core/src/main/java/org/apache/geode/cache/Region.java be70157 
>   geode-core/src/main/java/org/apache/geode/internal/cache/LocalRegion.java 
> 4349359 
>   geode-core/src/test/java/org/apache/geode/cache/query/RegionJUnitTest.java 
> 1ba858b 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/dunit/RemoteQueryDUnitTest.java
>  8088374 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/partitioned/PRInvalidQueryJUnitTest.java
>  6239b54 
> 
> Diff: https://reviews.apache.org/r/54436/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Jason Huynh
> 
>

Re: Top-Level Project Tasks Completed

[Jianxia Chen]

Great!

On Tue, Dec 6, 2016 at 10:34 AM, Mark Bretl  wrote:

> As an update, the GitHub mirror is online and available at
> https://github.com/apache/geode.
>
> --Mark
>
> On Mon, Dec 5, 2016 at 5:13 PM, Mark Bretl  wrote:
>
> > Hi Everyone,
> >
> > The Apache INFRA team has completed the Top-Level Project for Apache
> Geode.
> >
> > Summary of Changes:
> > - Website is https://geode.apache.org/
> > - Mailing lists (dev, user, private) now have suffix of '@
> geode.apache.org
> > '
> > - GitHub URL is now https://github.com/apache/geode (It has error of 404
> > now, but should resolve in about 7 hours)
> >
> > If you have any issues, please email the Dev list at '
> dev@geode.apache.org'
> > so we can investigate.
> >
> > Thanks to everyone for their patience. Now, let's move forward with
> > growing this community and completing another release!
> >
> > Best Regards,
> >
> > Mark Bretl
> > V.P. Apache Geode
> >
>

Re: Travis on github PRs

[Anthony Baker]

I have some changes queued up to the travis yml on GEODE-2156 (there may be 
something else to be done afterwards).

Anthony


> On Dec 6, 2016, at 11:36 AM, Kirk Lund  wrote:
> 
> Looks like the Travis integration on https://github.com/apache/geode needs
> to be updated. It still refers to incubator-geode and clicking the
> "Details" link goes to an error page "The repository at
> apache/incubator-geode was not found."
> 
> Does anyone who knows how to fix this want to fix it? Or feel free to send
> me info on what to do and I'll set aside time to learn how to do this (not
> sure I have permissions on github though).
> 
> Thanks,
> Kirk

Failed: apache/geode#1549 (GEODE-1912 - ce9e138)

[Travis CI]

Build Update for apache/geode
-

Build: #1549
Status: Failed

Duration: 5 minutes and 25 seconds
Commit: ce9e138 (GEODE-1912)
Author: Jinmei Liao
Message: GEODE-1912: use Spring shell's parser and delete our own parsing code

View the changeset: https://github.com/apache/geode/commit/ce9e138cc99d

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

--

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

Re: Review Request 54438: GEODE-2185: Index not used with parameterized query

[Barry Oglesby]

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


Ship it!




Ship It!

- Barry Oglesby


On Dec. 6, 2016, 7:03 p.m., Jason Huynh wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54438/
> ---
> 
> (Updated Dec. 6, 2016, 7:03 p.m.)
> 
> 
> Review request for geode and Barry Oglesby.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> Queries with parameterized values were not using indexes as expected.
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/main/java/org/apache/geode/cache/query/internal/CompiledBindArgument.java
>  cb9218b 
>   
> geode-core/src/main/java/org/apache/geode/cache/query/internal/IndexTrackingQueryObserver.java
>  a643b59 
>   geode-core/src/test/java/org/apache/geode/cache/query/data/Instrument.java 
> PRE-CREATION 
>   geode-core/src/test/java/org/apache/geode/cache/query/data/TradingLine.java 
> PRE-CREATION 
>   
> geode-core/src/test/java/org/apache/geode/cache/query/internal/index/MapRangeIndexMaintenanceJUnitTest.java
>  5159f89 
> 
> Diff: https://reviews.apache.org/r/54438/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Jason Huynh
> 
>

Build failed in Jenkins: Geode-release #36

[Apache Jenkins Server]

See 

--
[...truncated 56 lines...]
Seen branch in repository origin/feature/GEODE-1968
Seen branch in repository origin/feature/GEODE-1983
Seen branch in repository origin/feature/GEODE-1985
Seen branch in repository origin/feature/GEODE-1987
Seen branch in repository origin/feature/GEODE-2001
Seen branch in repository origin/feature/GEODE-2012
Seen branch in repository origin/feature/GEODE-2026
Seen branch in repository origin/feature/GEODE-2104
Seen branch in repository origin/feature/GEODE-2156
Seen branch in repository origin/feature/GEODE-2157
Seen branch in repository origin/feature/GEODE-217
Seen branch in repository origin/feature/GEODE-259
Seen branch in repository origin/feature/GEODE-288
Seen branch in repository origin/feature/GEODE-308
Seen branch in repository origin/feature/GEODE-33
Seen branch in repository origin/feature/GEODE-37
Seen branch in repository origin/feature/GEODE-37_1
Seen branch in repository origin/feature/GEODE-37_2
Seen branch in repository origin/feature/GEODE-417
Seen branch in repository origin/feature/GEODE-52
Seen branch in repository origin/feature/GEODE-523
Seen branch in repository origin/feature/GEODE-564
Seen branch in repository origin/feature/GEODE-574
Seen branch in repository origin/feature/GEODE-607
Seen branch in repository origin/feature/GEODE-610
Seen branch in repository origin/feature/GEODE-639
Seen branch in repository origin/feature/GEODE-64
Seen branch in repository origin/feature/GEODE-678
Seen branch in repository origin/feature/GEODE-681
Seen branch in repository origin/feature/GEODE-78
Seen branch in repository origin/feature/GEODE-781
Seen branch in repository origin/feature/GEODE-79
Seen branch in repository origin/feature/GEODE-80
Seen branch in repository origin/feature/GEODE-866
Seen branch in repository origin/feature/GEODE-917
Seen branch in repository origin/feature/GEODE-92
Seen branch in repository origin/feature/GEODE-93
Seen branch in repository origin/feature/GEODE-996
Seen branch in repository origin/feature/e2e-testing
Seen branch in repository origin/feature/spotlessPlugin
Seen branch in repository origin/feature/wan_single_hop_wip
Seen branch in repository origin/gh-wiki
Seen branch in repository origin/master
Seen branch in repository origin/native-client-software-grant
Seen branch in repository origin/sga2
Seen branch in repository origin/staging/docs-grant1
Seen branch in repository origin/wan_cq_donation
Seen 89 remote branches
ERROR: Couldn't find any revision to build. Verify the repository and branch 
configuration for this job.
Retrying after 10 seconds
 > git rev-parse --is-inside-work-tree # timeout=10
Fetching changes from the remote Git repository
 > git config remote.origin.url 
 > https://git-wip-us.apache.org/repos/asf/geode.git # timeout=10
Fetching upstream changes from https://git-wip-us.apache.org/repos/asf/geode.git
 > git --version # timeout=10
 > git -c core.askpass=true fetch --tags --progress 
 > https://git-wip-us.apache.org/repos/asf/geode.git 
 > +refs/heads/*:refs/remotes/origin/*
Seen branch in repository origin/GEODE-1372
Seen branch in repository origin/GEODE-1648
Seen branch in repository origin/GEODE-1883
Seen branch in repository origin/GEODE-1912
Seen branch in repository origin/GEODE-2153
Seen branch in repository origin/asf-site
Seen branch in repository origin/asf/asf-site
Seen branch in repository origin/develop
Seen branch in repository origin/feature/GEM-1032
Seen branch in repository origin/feature/GEODE-10
Seen branch in repository origin/feature/GEODE-1017
Seen branch in repository origin/feature/GEODE-1040
Seen branch in repository origin/feature/GEODE-1056
Seen branch in repository origin/feature/GEODE-1096
Seen branch in repository origin/feature/GEODE-11
Seen branch in repository origin/feature/GEODE-1109
Seen branch in repository origin/feature/GEODE-1153
Seen branch in repository origin/feature/GEODE-1186
Seen branch in repository origin/feature/GEODE-12
Seen branch in repository origin/feature/GEODE-1209
Seen branch in repository origin/feature/GEODE-1269
Seen branch in repository origin/feature/GEODE-1372
Seen branch in repository origin/feature/GEODE-1374
Seen branch in repository origin/feature/GEODE-14
Seen branch in repository origin/feature/GEODE-14-2
Seen branch in repository origin/feature/GEODE-1464
Seen branch in repository origin/feature/GEODE-1466
Seen branch in repository origin/feature/GEODE-1493
Seen branch in repository origin/feature/GEODE-1573
Seen branch in repository origin/feature/GEODE-16
Seen branch in repository origin/feature/GEODE-1673-PR-212
Seen branch in repository origin/feature/GEODE-17
Seen branch in repository origin/feature/GEODE-17-2
Seen branch in repository origin/feature/GEODE-178
Seen branch in repository origin/feature/GEODE-1861
Seen branch in repository origin/feature/GEODE-1883
Seen branch in repository origin/feature/GEODE-1895
Seen branch in 

Review Request 54438: GEODE-2185: Index not used with parameterized query

[Jason Huynh]

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

Review request for geode and Barry Oglesby.


Repository: geode


Description
---

Queries with parameterized values were not using indexes as expected.


Diffs
-

  
geode-core/src/main/java/org/apache/geode/cache/query/internal/CompiledBindArgument.java
 cb9218b 
  
geode-core/src/main/java/org/apache/geode/cache/query/internal/IndexTrackingQueryObserver.java
 a643b59 
  geode-core/src/test/java/org/apache/geode/cache/query/data/Instrument.java 
PRE-CREATION 
  geode-core/src/test/java/org/apache/geode/cache/query/data/TradingLine.java 
PRE-CREATION 
  
geode-core/src/test/java/org/apache/geode/cache/query/internal/index/MapRangeIndexMaintenanceJUnitTest.java
 5159f89 

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


Testing
---


Thanks,

Jason Huynh

Review Request 54436: GEODE-2183: Region.query behaves differently on client and server when full syntax is passed as a predicate

[Jason Huynh]

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

Review request for geode, Barry Oglesby, nabarun nag, and Dan Smith.


Repository: geode


Description
---

Added deconstruction/reconstruction code to server side region.query execution 
to make it consistent with client side.

At some point I think we were going to deprecate the ability to execute a full 
select query string from the region.query() method itself.

Minor prevention of executing a full query with a region that does not match 
the current region name and prevents multiple region queries from being 
executed from region.query.  Long term we should restrict this even more to 
prevent security issues.


Diffs
-

  geode-core/src/main/java/org/apache/geode/cache/Region.java be70157 
  geode-core/src/main/java/org/apache/geode/internal/cache/LocalRegion.java 
4349359 
  geode-core/src/test/java/org/apache/geode/cache/query/RegionJUnitTest.java 
1ba858b 
  
geode-core/src/test/java/org/apache/geode/cache/query/dunit/RemoteQueryDUnitTest.java
 8088374 
  
geode-core/src/test/java/org/apache/geode/cache/query/partitioned/PRInvalidQueryJUnitTest.java
 6239b54 

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


Testing
---


Thanks,

Jason Huynh

Re: TLP Transition Changes Coming

[Roman Shaposhnik]

Mark, you're one of the JIRA admins for the project -- if you can't do
it -- it means you need to file an INFRA ticket.

Thanks,
Roman.

On Tue, Dec 6, 2016 at 10:37 AM, Mark Bretl  wrote:
> Roman,
>
> It looks like JIRA notifications are still going to
> iss...@geode.incubator.apache.org and Geode is still listed as Incubator,
> would you be able to update it or is that a task for INFRA?
>
> --Mark
>
> On Mon, Dec 5, 2016 at 5:11 PM, Mark Bretl  wrote:
>
>> TLP Tasks are completed.
>>
>> New Apache Git URL: https://git.apache.org/geode.git
>> New GitHub Mirror URL: https://github.com/apache/geode
>>
>> Easiest way to migrate to the new URL is to edit your Git config. Looks
>> like the GitHub Mirror might take until later tonight, Pacifc Standard Time.
>>
>> If you run into any issues, be sure to email the dev list.
>>
>> --Mark
>>
>>
>>
>> On Mon, Dec 5, 2016 at 10:39 AM, Mark Bretl  wrote:
>>
>>> As stated last week, INFRA will be working on the TLP tasks today and I
>>> have been notified they will be starting within the hour.
>>>
>>> Best Regards,
>>>
>>> --Mark
>>>
>>>
>>>
>>> On Fri, Dec 2, 2016 at 12:44 PM, Joey McAllister 
>>> wrote:
>>>
 +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: Top-Level Project Tasks Completed

[Roman Shaposhnik]

Congrats!

Thanks,
Roman.

On Mon, Dec 5, 2016 at 5:11 PM, Mark Bretl  wrote:
> Hi Everyone,
>
> The Apache INFRA team has completed the Top-Level Project for Apache Geode.
>
> Summary of Changes:
> - Website is https://geode.apache.org/
> - Mailing lists (dev, users, private) now have suffix of '@geode.apache.org'
> - GitHub URL is now https://github.com/apache/geode (It has error of 404
> now, but should resolve in about 7 hours)
>
> If you have any issues, please email the Dev list at 'dev@geode.apache.org'
> so we can investigate.
>
> Thanks to everyone for their patience. Now, let's move forward with growing
> this community and completing another release!
>
> Best Regards,
>
> Mark Bretl
> V.P. Apache Geode

Re: Top-Level Project Tasks Completed

[Dor Ben Dov]

Thanks Mark

Dor

 Original Message 

Subject: Re: Top-Level Project Tasks Completed

From: Mark Bretl 

Date: 6 בדצמ' 2016, 09:39

To: dev@geode.apache.org

Hi Dor,

Changes to the website and other Incubator branded material are being
tracked as issue GEODE-2156 [1] and will be made in the near future.

Best Regards,

--Mark

[1] https://issues.apache.org/jira/plugins/servlet/mobile#issue/GEODE-2156

On Mon, Dec 5, 2016 at 10:54 PM Dor Ben Dov  wrote:

> Gregory,
>
> On the bottom of the page geode.apache.org there is still a sign that
> it's an incubation, are you going to remove this ?
>
>
>
> Regards,
>
> Dor
>
>
>
> -Original Message-
>
> From: Gregory Chase [mailto:gch...@pivotal.io]
>
> Sent: יום ג 06 דצמבר 2016 06:17
>
> To: dev@geode.apache.org
>
> Cc: us...@geode.apache.org
>
> Subject: Re: Top-Level Project Tasks Completed
>
>
>
> Yay! Congrats!
>
>
>
> On Mon, Dec 5, 2016 at 5:11 PM, Mark Bretl  wrote:
>
>
>
> > Hi Everyone,
>
> >
>
> > The Apache INFRA team has completed the Top-Level Project for Apache
> Geode.
>
> >
>
> > Summary of Changes:
>
> > - Website is https://geode.apache.org/
>
> > - Mailing lists (dev, users, private) now have suffix of '@
>
> > geode.apache.org'
>
> > - GitHub URL is now https://github.com/apache/geode (It has error of
>
> > 404 now, but should resolve in about 7 hours)
>
> >
>
> > If you have any issues, please email the Dev list at
>
> > 'dev@geode.apache.org '
>
> > so we can investigate.
>
> >
>
> > Thanks to everyone for their patience. Now, let's move forward with
>
> > growing this community and completing another release!
>
> >
>
> > Best Regards,
>
> >
>
> > Mark Bretl
>
> > V.P. Apache Geode
>
> >
>
>
>
>
>
>
>
> --
>
> Greg Chase
>
>
>
> Global Head, Big Data Communities
>
> http://www.pivotal.io/big-data
>
>
>
> Pivotal Software
>
> http://www.pivotal.io/
>
>
>
> 650-215-0477
>
> @GregChase
>
> Blog: http://geekmarketing.biz/
>
>
>
> This message and the information contained herein is proprietary and
> confidential and subject to the Amdocs policy statement,
>
> you may review at http://www.amdocs.com/email_disclaimer.asp
>
>

This message and the information contained herein is proprietary and 
confidential and subject to the Amdocs policy statement,
you may review at http://www.amdocs.com/email_disclaimer.asp