Skip to content

[HUDI-975] Add unit tests in TestHoodieTableFileSystemView to test view for non-partitioned table#1692

Merged
bvaradar merged 1 commit intoapache:masterfrom
cxzl25:fix_HUDI-975
Jun 1, 2020
Merged

[HUDI-975] Add unit tests in TestHoodieTableFileSystemView to test view for non-partitioned table#1692
bvaradar merged 1 commit intoapache:masterfrom
cxzl25:fix_HUDI-975

Conversation

@cxzl25
Copy link
Contributor

@cxzl25 cxzl25 commented May 31, 2020

What is the purpose of the pull request

Add unit tests in TestHoodieTableFileSystemView to test view for non-partitioned table

Brief change log

Add unit tests in TestHoodieTableFileSystemView to test view for non-partitioned table

Verify this pull request

This change added tests.

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.

@codecov-commenter
Copy link

codecov-commenter commented May 31, 2020

Codecov Report

Merging #1692 into master will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1692   +/-   ##
=========================================
  Coverage     18.19%   18.20%           
- Complexity      856      857    +1     
=========================================
  Files           348      348           
  Lines         15344    15344           
  Branches       1523     1523           
=========================================
+ Hits           2792     2793    +1     
+ Misses        12195    12193    -2     
- Partials        357      358    +1     
Impacted Files Coverage Δ Complexity Δ
...che/hudi/common/table/timeline/TimelineLayout.java 85.71% <0.00%> (-7.15%) 3.00% <0.00%> (ø%)
...apache/hudi/common/fs/HoodieWrapperFileSystem.java 22.69% <0.00%> (+0.70%) 29.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1a9fef4...400df91. Read the comment docs.

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 good.

@bvaradar bvaradar merged commit 7c59095 into apache:master Jun 1, 2020
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