cbalci commented on PR #13080: URL: https://github.com/apache/pinot/pull/13080#issuecomment-2094657498
Note: Implementing this inside the `TypeConverter` looks a bit out of place, but I think it is because the class is somewhat undernamed. It is more like an 'extractor' which is responsible for converting `DataTable` stream into a Spark Row stream, so seemed to me as the right abstraction. -- 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
