Re: Review Request 42656: Don't destroy session between requests with TRequestsTransport

2016-03-08 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42656/#review122620 --- I am fine with this change and will commit it this way. I'll just

Re: Review Request 42656: Don't destroy session between requests with TRequestsTransport

2016-03-07 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42656/#review122367 --- Ship it! Master (a91a759) is green with this patch. ./build-s

Re: Review Request 42656: Don't destroy session between requests with TRequestsTransport

2016-03-07 Thread Kunal Thakar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42656/ --- (Updated March 7, 2016, 7:38 p.m.) Review request for Aurora and Stephan Erb.

Re: Review Request 42656: Don't destroy session between requests with TRequestsTransport

2016-02-18 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42656/#review119634 --- The tests are failing with my proposed change. I will have to re-c

Re: Review Request 42656: Don't destroy session between requests with TRequestsTransport

2016-02-17 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42656/#review119561 --- Master (9d642dc) is red with this patch. ./build-support/jenkins

Re: Review Request 42656: Don't destroy session between requests with TRequestsTransport

2016-02-17 Thread Kunal Thakar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42656/ --- (Updated Feb. 18, 2016, 12:01 a.m.) Review request for Aurora and Stephan Erb.

Re: Review Request 42656: Don't destroy session between requests with TRequestsTransport

2016-02-16 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42656/#review119328 --- Fix it, then Ship it! I ran some tests. I have dropped the open

Re: Review Request 42656: Don't destroy session between requests with TRequestsTransport

2016-01-22 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42656/#review115961 --- Would be great if you could also run the end-to-end tests to verif

Re: Review Request 42656: Don't destroy session between requests with TRequestsTransport

2016-01-22 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42656/#review115870 --- Ship it! Master (66a4d5f) is green with this patch. ./build-supp

Review Request 42656: Don't destroy session between requests with TRequestsTransport

2016-01-22 Thread Kunal Thakar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42656/ --- Review request for Aurora. Repository: aurora Description --- As an API