Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Using FsUtils instead of Files API to extract file extension #213

Merged
merged 1 commit into from
Jun 30, 2017

Conversation

n3nash
Copy link
Contributor

@n3nash n3nash commented Jun 29, 2017

This PR solves #212

@n3nash n3nash requested a review from prazanna June 29, 2017 23:44
@prazanna
Copy link
Contributor

Will merge in once Travis builds

@n3nash
Copy link
Contributor Author

n3nash commented Jun 29, 2017

Looking into the failures.

@n3nash n3nash force-pushed the hoodieinstant_bug_fix branch from 9f671f1 to f9ad1f8 Compare June 30, 2017 00:09
@n3nash
Copy link
Contributor Author

n3nash commented Jun 30, 2017

@prazanna Can you take a look at the diff again ? I made some minor changes not sure if you got a chance to look at that before approving.

@prazanna prazanna merged commit 348250d into apache:master Jun 30, 2017
@n3nash n3nash deleted the hoodieinstant_bug_fix branch June 30, 2017 04:08
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants