Re: Review Request 45131: GEODE-923 Validation for idle Timeout set properly.

2016-03-21 Thread Bruce Schuchardt

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


Ship it!




Ship It!

- Bruce Schuchardt


On March 21, 2016, 11:19 p.m., Hitesh Khamesra wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45131/
> ---
> 
> (Updated March 21, 2016, 11:19 p.m.)
> 
> 
> Review request for geode, Bruce Schuchardt, Jianxia Chen, Udo Kohlmeyer, and 
> Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> Now before making connection passivate, we start time and then wait
> until connection has been destroy. And making sure it remains idle
> for idle timeout. Added 100 ms padding as well.
> 
> 
> Diffs
> -
> 
>   
> geode-core/src/test/java/com/gemstone/gemfire/cache/client/internal/pooling/ConnectionManagerJUnitTest.java
>  d0b2991 
> 
> Diff: https://reviews.apache.org/r/45131/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Hitesh Khamesra
> 
>



Review Request 45131: GEODE-923 Validation for idle Timeout set properly.

2016-03-21 Thread Hitesh Khamesra

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

Review request for geode, Bruce Schuchardt, Jianxia Chen, Udo Kohlmeyer, and 
Dan Smith.


Repository: geode


Description
---

Now before making connection passivate, we start time and then wait
until connection has been destroy. And making sure it remains idle
for idle timeout. Added 100 ms padding as well.


Diffs
-

  
geode-core/src/test/java/com/gemstone/gemfire/cache/client/internal/pooling/ConnectionManagerJUnitTest.java
 d0b2991 

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


Testing
---


Thanks,

Hitesh Khamesra