-
Notifications
You must be signed in to change notification settings - Fork 3.8k
feat: Bounded Stream Supervisor #19372
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 45 commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
dfda88c
Initial implementation for BoundedStreamConfig
aho135 4180ace
Implement isOffsetAtOrBeyond for Rabbit and Kinesis
aho135 8cb75f6
Unit test coverage
aho135 300ebe3
Fix BoundedStreamConfigTest
aho135 9af9729
Remove unused import
aho135 e0ffef6
Remove unneeded tests
aho135 162e1f3
Unit test fix
aho135 3ea2b0b
Fix import and add coverage for RabbitStreamSupervisor
aho135 8e3e81c
Test coverage for validateBoundedStreamConfig
aho135 4bed658
Re-initialize partition group and reset state after reset
aho135 c9181f0
Handle edge case where startOffset equals endOffset
aho135 9e85331
Compare Kinesis sequence numbers using BigInteger
aho135 9a32ce0
Remove stale test case
aho135 b04e907
Remove redundant validation of boundedStreamConfig
aho135 8e6dfb8
Throw DruidException with ADMIN persona for BoundedStreamConfig
aho135 f03abb6
Clean up unused Logger
aho135 ae9083f
javadoc and comment cleanup for isBoundedWorkComplete
aho135 f8a313b
Add embedded test for bounded ingestion
aho135 5965ac4
Add boundedStreamConfig to SeekableStreamDataSourceMetadata for metad…
aho135 9e66948
Revert pendingCompletionGroups check
aho135 902e118
Unit test fix
aho135 670749c
embedded-test for metadata mismatch
aho135 2457caf
Remove unused var
aho135 3943ad0
Unit test fix
aho135 395fa9a
Add boundedStreamConfig documentation
aho135 4cde39f
Fix spellcheck
aho135 7e86ec6
Increase code coverage
aho135 d985dc4
Increase coverage for BoundedStreamConfig
aho135 021e721
Remove unnecessary test
aho135 d23d9c4
Simplify completion check in createNewTasks
aho135 42ada89
Remove unused function
aho135 ed589c2
Unit test bounded supervisor completion
aho135 234bc82
Improve coverage on RabbitStreamSupervisor
aho135 1cd928d
Unit test coverage
aho135 094427d
Unit test for IllegalArgumentException for KafkaSupervisor
aho135 3ad278d
Check if end offsets are exclusive for bounded work completion
aho135 cf623b8
Increase branch coverage
aho135 a037ccb
Increase branch coverage
aho135 0e6466c
Unit test coverage
aho135 b1b1179
Fix import
aho135 126638f
Remove use of deprecated function
aho135 2ff9cfd
Revert to deprecated function since not initialized in mock object
aho135 2d42ab4
Merge branch 'master' into bounded-stream-supervisor
aho135 31c870e
Fix merge conflict
aho135 b68705a
Detect metadata mismatch when committed offset > bounded config end
aho135 6c33c8f
Clean up redundant tests in BoundedStreamConfigTest and use EqualsVer…
aho135 41a31a1
Compare Kinesis Sequence numbers using built in comparison
aho135 dd9bdd2
Clean up docs based on review comments
aho135 fd7981f
Early return before convert for hasTaskGroupReachedBoundedEnd
aho135 5bc87fb
Merge branch 'master' into bounded-stream-supervisor
aho135 cc13428
Resolve merge conflicts
aho135 c62b936
Fix KinesisSupervisorTest
aho135 35c4574
Update KinesisSupervisorTest.java
aho135 e0b5ef9
Cover case where start > end
aho135 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.