-
Notifications
You must be signed in to change notification settings - Fork 288
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
better lightning-loader configuration #3519
Comments
An alternative for feature 1 is we put |
I put it under target-database because, if a task allows multiple targets in the future, it may be possible to use different import-mode for different targets |
cc @glorv |
Since the |
from the functionality, this field should be put under But we should also make sure this field is related to downstream and |
Is your feature request related to a problem?
currently DM's config for lightning loader is not well-designed
Describe the feature you'd like
Regarding the new backend configuration item, I hope it could be optimized.
Since this configuration is related to target, I think it can be placed in
tidb/local
could be replaced withlogical/physical
orsql/sst
?,looks better to understandDescribe alternatives you've considered
No response
Teachability, Documentation, Adoption, Migration Strategy
No response
The text was updated successfully, but these errors were encountered: