Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Kafka >> mail # dev >> Re: Review Request 18343: Patch for KAFKA-1250

Copy link to this message
Re: Review Request 18343: Patch for KAFKA-1250

This is an automatically generated e-mail. To reply, visit:
1. Is there a reason that you want to change the license header? We should stay with one format of the license header.


    I am still confused here. Could you clarify when we should add a matching {} for exception? The usage in line 89 and line 77 is not consistent. My understanding is that if exception is the only parameter, you don't need to add the {}. Otherwise, you will. Is that correct?


    Are we missing a {}?


    From the mailing list discussion, most people prefer to see this in info (at least the overridden values) level.
- Jun Rao
On Feb. 26, 2014, 5:32 a.m., Jay Kreps wrote: