-
Notifications
You must be signed in to change notification settings - Fork 2.5k
[HUDI-7399] hudi-aws sync integration testing #10614
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
Conversation
801379f to
c555788
Compare
5c00e45 to
d29fd4e
Compare
keep a copy of data generator, to be adapted
| <version>${project.version}</version> | ||
| </dependency> | ||
|
|
||
| <!-- Hadoop --> |
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.
@parisni : has this change been tested with real glue and emr setup?
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.
not yet. Do you have insight why hadoop/hive depts were added ?
yihua
left a comment
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.
@parisni Do you still plan to work on and update this PR? Adding test coverage is always good.
|
done in #14234 |
Change Logs
Impact
None
Risk level (write none, low medium or high below)
None
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change
ticket number here and follow the instruction to make
changes to the website.
Contributor's checklist