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.
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
feat: Configurable batch size and max wait limit for targets #1876
base: main
Are you sure you want to change the base?
feat: Configurable batch size and max wait limit for targets #1876
Changes from 78 commits
11c56e3
550ecff
68c3484
66f750e
62c2c32
541a46d
028e3d9
4309c8e
dc2a2d8
01f4539
8a591c1
eb6c9e7
f6bbf0c
74b1653
bb0c8fb
c151f84
49c014c
a425774
379f814
0303f25
1fa9752
d2db4fb
99a9fb5
059c392
7ac5a6c
0420db8
4b5b3c6
1aec608
b5e93a6
57b7f37
f5874c9
3b73e64
b48f3d0
6c465f7
3cee381
7025280
8bbb346
fd8f171
2cfa516
da2808a
01962e0
c89fd74
2f51fd4
bf582e0
01705c6
29ae3de
1b5cc6a
3d19a03
50d00c2
fb5d6f2
e574003
7f48881
7a54c69
1261c04
0436a74
b92d8c1
f78cc0a
ec57105
68a520c
b36acf8
933fa21
c342e44
9ce87da
af84ecd
31f6ef5
9dae576
b2697ee
e874f73
f36a79f
2952513
c79cf64
4f75636
879995a
1c81669
7a1df1d
c804b76
648656b
dc8481d
ffabffa
50e82ae
f977ca2
b2181e7
484c7cb
2a870af
fd787f2
e159ae6
7c4756c
e50f1a2
f4c2c38
5c1b5ea
39dd6e7
ab2782d
d06481d
295802f
17f143c
a0e34c9
49a64e6
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm still finding this a bit confusing. I think we want the max batch size to have a default value, and the max wait time to default to
None
so that in the default case, only the current batch size is checked.Essentially:
Does it make sense? I'm sorry, I feel like I'm missing something 🙏
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The diagram makes sense. I will need to work through it again tomorrow. I think you feeling like you are missing something is a result of my poor writing. Below is the best flowchart for
batch_dynamic_management
I could make.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This would be when
batch_dynamic_management
is set toTrue
in the target config andbatch_size_rows
andbatch_wait_limit_seconds
are not preset in the target config?or
Are you meaning default as in none of the three target config settings
batch_size_rows
,batch_wait_limit_seconds
, andbatch_dynamic_management
are present?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@edgarrmondragon When you have a free moment would you please provide me clarification on what you see the "default case" to be. 🙏😃
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you the other setting here?