lw309637554 commented on pull request #1810:
URL: https://github.com/apache/hudi/pull/1810#issuecomment-658740024
@vinothchandar @garyli1019 I think in this PR the hudi-sync abstract is
ready. Expect your review. Thanks
1. the module abstract is
hudi-sync
hudi-hive-sync/
hudi-dla-sync/
2. about backwards compatible .
a. about users code compatible: because do not modify the hivesync class
name. Users do not need to modify these old code.
b. about users pom dependency compatible: because do not modify the module
name hudi-hive-sync. Users not need to modify pom dependency
c. about users local jar files compatible: if users's local jars is do not
shaded the hudi-hive-sync's
Indirect dependence. Old users just need add the hudi-sync-common.jar to
their directory. Just like hudi-utilities-bundle modiry in this PR.
d. about sync parameters, just add new parameters such as --enable-sync as
new default parameter,also backwards compatible --enable-hive-sync.
3. some others works ,such as update the doc. will do in other issues.
what about your suggestion?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
