-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Support Rsync to transfer clickhouse data file #2074
Conversation
@Hisoka-X Please have a look, and give me some advice.Thank you! |
<dependency> | ||
<groupId>com.github.fracpete</groupId> | ||
<artifactId>rsync4j-all</artifactId> | ||
<version>3.2.3-10</version> | ||
</dependency> |
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.
https://mvnrepository.com/artifact/com.github.fracpete/rsync4j-all
The Apache License and GPL are not compatible and we cannot use this dependency
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.
Thank you. I will try to use the buildin package scala.sys.process to build rsync shell command to realize this feature. And recommit it.
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.
Thank you. I will try to use the buildin package scala.sys.process to build rsync shell command to realize this feature. And recommit it.
After you change, ping me. thx
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.
@Hisoka-X i have changed and passed the simple test in spark-local mode , please check it.
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, well done!
Can you submit this feature to our new seatunnel api connector? If so, it's very wonderful. |
Sure, I'm happy to do it. |
Purpose of this pull request
#2073
Support Rsync to transfer clickhouse data file.