abhishekrb19 opened a new pull request, #17151: URL: https://github.com/apache/druid/pull/17151
Follow up to https://github.com/apache/druid/pull/17137. Instead of moving the streaming-only methods to the `SeekableStreamSupervisor` abstract class, this patch moves them to a separate `StreamSupervisor` interface. The reason is that the `SeekableStreamSupervisor` abstract class also has many other abstract methods. The `StreamSupervisor` interface on the other hand provides a minimal set of functions offering a good middle ground for any custom concrete implementation that doesn't require all the goodies from `SeekableStreamSupervisor`. Changes: - Move the methods that was moved in https://github.com/apache/druid/pull/17137 to a separate `StreamSupervisor` interface - Update the `instanceof` checks in `SupervisorManager` to reflect the more generic interface `StreamSupervisor` -- 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...@druid.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
