Re: Review Request 26018: Patch for KAFKA-404

2014-09-25 Thread Jonathan Creasy
I went and looked at the ServerShutdownTest to implement your suggestion, and I don¹t think I like this particular consolidation. How about I go and change this test to ServerStartupTest, and future startup checks can go there and we leave the Shutdown checks separate? On 9/25/14, 12:55 PM, "Neha

Re: Review Request 26018: Patch for KAFKA-404

2014-09-25 Thread Neha Narkhede
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26018/#review54559 --- core/src/test/scala/unit/kafka/server/CreateZKChrootTest.scala