Skip to content
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

[HUDI-68] Pom cleanup & demo automation #846

Merged

Conversation

vinothchandar
Copy link
Member

No description provided.

@vinothchandar
Copy link
Member Author

@thesuperzapper this is now up, with full demo automation.. FYI

@vinothchandar vinothchandar changed the title Pom cleanup & demo automation [HUDI-62] Pom cleanup & demo automation Aug 22, 2019
@@ -127,6 +127,8 @@
<pattern>parquet.schema</pattern>
<shadedPattern>org.apache.hudi.parquet.schema</shadedPattern>
</relocation>
<!-- TODO: Revisit GH ISSUE #533 & PR#633-->
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bvaradar this needs your special attention.. Had to do this to get hive sync as a part of spark datasource working.. some background in those issues and prs

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack. We would have to check with other certified combinations of Hive and Spark

@vinothchandar vinothchandar changed the title [HUDI-62] Pom cleanup & demo automation [HUDI-68] Pom cleanup & demo automation Aug 22, 2019
@vinothchandar
Copy link
Member Author

CI has passed 4 times in a row now. I think its stable enough to merge

Copy link
Contributor

@bvaradar bvaradar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks overall good. Really like the optimizations around batching beeline commands. Should keep integration tests faster

@@ -127,6 +127,8 @@
<pattern>parquet.schema</pattern>
<shadedPattern>org.apache.hudi.parquet.schema</shadedPattern>
</relocation>
<!-- TODO: Revisit GH ISSUE #533 & PR#633-->
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack. We would have to check with other certified combinations of Hive and Spark

thesuperzapper and others added 3 commits August 22, 2019 19:45
  - Fix ordering of dependencies in poms, to enable better resolution
  - Idea is to place more specific ones at the top
  - And place dependencies which use them below them
 - Move hive queries from hive cli to beeline
 - Standardize on taking query input from text command files
 - Deltastreamer ingest, also does hive sync in a single step
 - Spark Incremental Query materialized as a derived Hive table using datasource
 - Fix flakiness in HDFS spin up and output comparison
 - Code cleanup around streamlining and loc reduction
 - Also fixed pom to not shade some hive classs in spark, to enable hive sync
@vinothchandar vinothchandar merged commit 6edf0b9 into apache:master Aug 23, 2019
bvaradar pushed a commit to bvaradar/hudi that referenced this pull request Sep 19, 2019
 - [HUDI-172] Cleanup Maven POM/Classpath
  - Fix ordering of dependencies in poms, to enable better resolution
  - Idea is to place more specific ones at the top
  - And place dependencies which use them below them
- [HUDI-68] : Automate demo steps on docker setup
 - Move hive queries from hive cli to beeline
 - Standardize on taking query input from text command files
 - Deltastreamer ingest, also does hive sync in a single step
 - Spark Incremental Query materialized as a derived Hive table using datasource
 - Fix flakiness in HDFS spin up and output comparison
 - Code cleanup around streamlining and loc reduction
 - Also fixed pom to not shade some hive classs in spark, to enable hive sync
@yanghua yanghua mentioned this pull request Jan 27, 2020
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants