Skip to content

[MINOR] Remove unused directory - #1190

Merged
xianjingfeng merged 1 commit into
apache:masterfrom
roryqi:minor
Sep 8, 2023
Merged

[MINOR] Remove unused directory#1190
xianjingfeng merged 1 commit into
apache:masterfrom
roryqi:minor

Conversation

@roryqi

@roryqi roryqi commented Sep 7, 2023

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Remove the unused directory.

Why are the changes needed?

We support Tez framework without extra patch.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

CI passed.

@roryqi
roryqi requested a review from xianjingfeng September 7, 2023 11:37
@codecov-commenter

codecov-commenter commented Sep 7, 2023

Copy link
Copy Markdown

Codecov Report

Merging #1190 (c2d6334) into master (a3e2c91) will increase coverage by 1.08%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1190      +/-   ##
============================================
+ Coverage     53.65%   54.74%   +1.08%     
- Complexity     2581     2584       +3     
============================================
  Files           391      371      -20     
  Lines         22409    20049    -2360     
  Branches       1875     1875              
============================================
- Hits          12024    10975    -1049     
+ Misses         9676     8439    -1237     
+ Partials        709      635      -74     

see 26 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@roryqi roryqi changed the title [MINOR] Remove extra directory [MINOR] Remove unused directory Sep 8, 2023

@xianjingfeng xianjingfeng left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@xianjingfeng
xianjingfeng merged commit f793a22 into apache:master Sep 8, 2023
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