Jackie-Jiang commented on PR #7358: URL: https://github.com/apache/pinot/pull/7358#issuecomment-1098592692
@ddcprg You may take a look at `HybridClusterIntegrationTest` on how to feed the data into both batch flow and streaming flow. In order to add an integration test, we need to add a new avro data set with the logical types. IMO it might be too much for this support (adding an integration test can add 1-2 minutes to the CI). A thorough unit test on avro record extractor should be good enough. -- 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
