Skip to content

Conversation

@sbernauer
Copy link
Contributor

What is the purpose of the pull request

Add some testcases for schema evolution. Reproduce a problem we faced.

Verify this pull request

This change added tests and can be verified as follows:

mvn -DskipITs -Dscala-2.12 -DfailIfNoTests=false -Dtest=TestHoodieDeltaStreamer#testSchemaEvolution clean test

Committer checklist

  • Has a corresponding JIRA in PR title & commit

  • Commit message is descriptive of the change

  • CI is green

  • Necessary doc changes done or have another open PR

  • For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@vinothchandar vinothchandar added the status:in-progress Work in progress label Jul 29, 2020
@vinothchandar
Copy link
Member

cc @codope can you take care of this? i.e either add as valid test case (may be it passed now with the recent fixes) or consider this part of the new design

@hudi-bot
Copy link
Collaborator

hudi-bot commented Sep 7, 2021

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run travis re-run the last Travis build
  • @hudi-bot run azure re-run the last Azure build

@vinothchandar vinothchandar added the area:schema Schema evolution and data types label Sep 7, 2021
@codope
Copy link
Member

codope commented Sep 10, 2021

cc @codope can you take care of this? i.e either add as valid test case (may be it passed now with the recent fixes) or consider this part of the new design

@vinothchandar This PR can be closed. I have included the tests and verified that the master works for those scenarios.

@sbernauer
Copy link
Contributor Author

Thanks @codope for your work!
The most relevant part of the tests got included with #2927
Thanks for adding the other part!

@sbernauer sbernauer closed this Sep 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:schema Schema evolution and data types status:in-progress Work in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants