ableegoldman commented on pull request #9821:
URL: https://github.com/apache/kafka/pull/9821#issuecomment-822625233


   Fine with me. It does seem odd that we'd be inconsistent, but I don't think 
we need to solve everything with this one KIP/PR.  Let's just file a ticket for 
this so we don't forget.
   
   (FWIW I think it was neither an oversight nor an intentional design, at 
least in the case of `queryMetadataForKey`, since this KIP-535 was started and 
completed while this KIP was still in progress, and the author most likely 
wasn't aware that we had plans to improve the IQ exceptions.)
   
   I think in that case, this PR covers everything it needs to. @vitojeng Can 
you just add a quick note about this new exception to the streams upgrade guide 
under the section for API changes in 3.0?
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to