mynameborat commented on PR #1631:
URL: https://github.com/apache/samza/pull/1631#issuecomment-1255408139

   > 
   
   Just noticed the constructor this is the partition key and not the message 
key. It should be okay from that front. In terms of the assumption on 
diagnostics stream is stream with one partition count, can that be created 
explicitly by the user? Or potentially expanded to more than one partition?
   
   In that case, it could still mean a incompatible change right?


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@samza.apache.org

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

Reply via email to