Skip to content

[HUDI-996] Add functional test suite in hudi-client#1824

Merged
yanghua merged 1 commit intoapache:masterfrom
xushiyan:add-functional-test-suite-to-hudi-client
Jul 15, 2020
Merged

[HUDI-996] Add functional test suite in hudi-client#1824
yanghua merged 1 commit intoapache:masterfrom
xushiyan:add-functional-test-suite-to-hudi-client

Conversation

@xushiyan
Copy link
Member

  • Add functional test suite in hudi-client
  • Tag TestHBaseIndex as functional

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.

@xushiyan xushiyan force-pushed the add-functional-test-suite-to-hudi-client branch 2 times, most recently from 0236947 to 7ff5dcc Compare July 13, 2020 23:52
@yanghua yanghua self-assigned this Jul 14, 2020
@yanghua yanghua self-requested a review July 14, 2020 01:08
@xushiyan
Copy link
Member Author

@yanghua will rebase this one once #1818 merged. thanks

- Add functional test suite in hudi-client
- Tag TestHBaseIndex as functional
@xushiyan xushiyan force-pushed the add-functional-test-suite-to-hudi-client branch from 148887a to 1ba1484 Compare July 14, 2020 03:48
Copy link
Contributor

@yanghua yanghua left a comment

Choose a reason for hiding this comment

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

LGTM

@yanghua
Copy link
Contributor

yanghua commented Jul 14, 2020

@vinothchandar Do you still have any concerns?

@yanghua yanghua merged commit b399b4a into apache:master Jul 15, 2020
@vinothchandar
Copy link
Member

No all good.. was waitng for you actually :)

@xushiyan xushiyan deleted the add-functional-test-suite-to-hudi-client branch July 25, 2020 22:50
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.

3 participants