KKcorps commented on PR #14311: URL: https://github.com/apache/pinot/pull/14311#issuecomment-2441299598
@Jackie-Jiang 14309 won't handle the case where .hasNext() itself fails. I understand this is backward incompatible. I think the main thing we want to do is use lineIterator by default instead of one from csv lib. -- 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...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org