-
Notifications
You must be signed in to change notification settings - Fork 132
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
compatible with the old version and take effect of dispatch and causality #915
Conversation
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.
rest LGTM
// if both setted, the disable one take affect. | ||
DisableDispatchFlag *bool `toml:"disable-dispatch-flag" json:"disable-dispatch-flag"` | ||
EnableDispatchFlag *bool `toml:"enable-dispatch-flag" json:"enable-dispatch-flag"` | ||
DisableDispatchFile *bool `toml:"disable-dispatch" json:"disable-dispatch"` |
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.
how about use these configs in integration test
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.
add in 505451b
LGTM |
@GregoryIan PTAL |
/run-all-tests |
/run-unit-test |
@csuzhangxc PTAL |
I'll task a look later today. |
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.
rest LGTM
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.
LGTM
/run-all-tests |
cherry pick to release-3.0 failed |
What problem does this PR solve?
disable-detect
anddisable-causality
merge
option inLoader
as default (the previous effect is false actually)What is changed and how it works?
EnableDispatch
andEnableCausality
forpkg/Loader
Check List
Tests
Code changes
Side effects
Related changes