[GitHub] [incubator-pinot] mcvsubbu commented on issue #4618: Make Kafka offset out of range as an transient exception.

2019-10-25 Thread GitBox
mcvsubbu commented on issue #4618: Make Kafka offset out of range as an 
transient exception.
URL: https://github.com/apache/incubator-pinot/pull/4618#issuecomment-546458541
 
 
   A repeated transient exception is eventually turned into a permanent 
exception, so I am fine with this. Can you please re-start the tests and get 
them to pass? We can commit this.


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


With regards,
Apache Git Services

-
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org



[GitHub] [incubator-pinot] mcvsubbu commented on issue #4618: Make Kafka offset out of range as an transient exception.

2019-09-17 Thread GitBox
mcvsubbu commented on issue #4618: Make Kafka offset out of range as an 
transient exception.
URL: https://github.com/apache/incubator-pinot/pull/4618#issuecomment-532377804
 
 
   Your first point is wrong. The validation manager will put in newer offsets 
that are valid before restarting. No manual fix is needed.


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


With regards,
Apache Git Services

-
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org