Hmm.  I'm not sure that "illegal" is that different from "invalid"
semantically.  If an argument is illegal for a given function, you could
also say that argument is invalid for that function.  Also, changing
exception names could be frustrating to users from a compatibility
perspective.  Perhaps some better documentation could help here?

best,
Colin


On Thu, Feb 16, 2017, at 16:32, Cosmin Lehene wrote:
> Wow, I managed to send two incomplete messages thanks to the new mac
> touchbar :)
> 
> 
> I was suggesting either renaming to IllegalTopicException or have a one
> that clearly mentions the name InvalidTopicNameException. However I guess
> the first option would be the easiest.
> 
> 
> Thanks,
> 
> Cosmin
> 
> 
> ________________________________
> From: Cosmin Lehene
> Sent: Thursday, February 16, 2017 4:29:14 PM
> To: dev@kafka.apache.org
> Subject: Should InvalidTopicException be called IllegalTopicException
> 
> 
> It's a bit confusing when trying to understand what's the difference
> between UnknownTopicOrPartitionException (retriable) and
> InvalidTopicException (non-retriable).
> 
> 
> Looking through the code to figure out what's an invalid topic I see most
> tests that throw it describe it as "illegal". This is also the case when
> trying to write to an internal topic.
> 
> 
> While a longer discussion around having an invalid topic _name
> 

Reply via email to