Re: Review Request 41266: GEODE-654: Changes LIFO queue to remove touched entry

2015-12-11 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41266/#review110039 ---

[GitHub] incubator-geode pull request: Feature/geode 663: adding more secur...

2015-12-11 Thread jinmeiliao
GitHub user jinmeiliao opened a pull request: https://github.com/apache/incubator-geode/pull/58 Feature/geode 663: adding more security test code into open source You can merge this pull request into a Git repository by running: $ git pull

Re: Review Request 41266: GEODE-654: Changes LIFO queue to remove touched entry

2015-12-11 Thread Vince Ford
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41266/ --- (Updated Dec. 12, 2015, 12:17 a.m.) Review request for geode, Darrel Schneider

Review Request 41274: Test hook beSick was sending response to final check. We should not be sending that to behave like sick member.

2015-12-11 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41274/ --- Review request for geode and Bruce Schuchardt. Repository: geode Description

Re: Review Request 41266: GEODE-654: Changes LIFO queue to remove touched entry

2015-12-11 Thread Vince Ford
> On Dec. 11, 2015, 11:36 p.m., Darrel Schneider wrote: > > gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/lru/NewLIFOClockHand.java, > > line 85 > > > > > > I find the continue block and the

[GitHub] incubator-geode pull request: Feature/geode 663: adding more secur...

2015-12-11 Thread upthewaterspout
Github user upthewaterspout commented on the pull request: https://github.com/apache/incubator-geode/pull/58#issuecomment-164080920 +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

Re: Review Request 41266: GEODE-654: Changes LIFO queue to remove touched entry

2015-12-11 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41266/#review110053 --- Ship it! Ship It! - Darrel Schneider On Dec. 11, 2015, 4:17

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

2015-12-11 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #155 was successful. --- Scheduled 1231 tests in total. https://build.spring.io/browse/SGF-NAG-155/ -- This

Re: Hung rat

2015-12-11 Thread Dick Cavender
I think it may matter if it's run right after a clean otherwise it tries to walk down into the build artifacts. It might be best to do a clean and then run rat before building. -Dick On 12/11/2015 5:52 PM, Dan Smith wrote: My build was hanging on the rat step - I found that by excluding the

Hung rat

2015-12-11 Thread Dan Smith
My build was hanging on the rat step - I found that by excluding the tags file it now works again. If you are still having problems with rat do a pull and see if it works for you now. -Dan

Review Request 41266: GEODE-654: Changes LIFO queue to remove touched entry

2015-12-11 Thread Vince Ford
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41266/ --- Review request for geode, Darrel Schneider and Dan Smith. Repository: geode

[GitHub] incubator-geode pull request: GEODE-655 Website sentence needs imp...

2015-12-11 Thread karensmolermiller
GitHub user karensmolermiller opened a pull request: https://github.com/apache/incubator-geode/pull/55 GEODE-655 Website sentence needs improvement The first sentence on the http://geode.incubator.apache.org/ web page means something different than intended. The sentence says

Review Request 41261: Removing TCPConduit's Stub ID class

2015-12-11 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41261/ --- Review request for geode and Darrel Schneider. Repository: geode Description

Re: Review Request 41261: Removing TCPConduit's Stub ID class

2015-12-11 Thread Jianxia Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41261/#review109982 --- Ship it! As long as there is no backward compatibility issue,

[GitHub] incubator-geode pull request: Feature/gem 164: move security tests...

2015-12-11 Thread jinmeiliao
GitHub user jinmeiliao opened a pull request: https://github.com/apache/incubator-geode/pull/56 Feature/gem 164: move security tests from closed to open and remove pivotal license and use ASF license …ompilation erros Closes #34 (cherry picked from commit

[GitHub] incubator-geode pull request: Feature/gem 164: move security tests...

2015-12-11 Thread jinmeiliao
Github user jinmeiliao closed the pull request at: https://github.com/apache/incubator-geode/pull/56 --- 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