Skip to content

adding support for Viewfs scheme#163

Closed
n3nash wants to merge 3 commits intoapache:masterfrom
n3nash:viewfs-change
Closed

adding support for Viewfs scheme#163
n3nash wants to merge 3 commits intoapache:masterfrom
n3nash:viewfs-change

Conversation

@n3nash
Copy link
Copy Markdown
Contributor

@n3nash n3nash commented May 18, 2017

When the default URI has viewfs (or the configuration object has viewfs URI), hoodie throws unsupported scheme exception. Adding this scheme to fix that issue.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 18, 2017

CLA assistant check
All committers have signed the CLA.

@vinothchandar
Copy link
Copy Markdown
Member

Thanks for the contribution! We are just looking into this at Uber for viewfs as well :). great timing.

BTW can you please sign the CLA, that will help us merge this in.

cc @ovj can you test this out for our ingestion system as well..

@vinothchandar
Copy link
Copy Markdown
Member

Also can you please merge your commits into a single one.. 3 commits feels excessive for this change

@n3nash
Copy link
Copy Markdown
Contributor Author

n3nash commented May 18, 2017

This pull request was accidentally made against spark 1.5.1, will close this and open a new one.

@n3nash n3nash closed this May 18, 2017
@n3nash n3nash deleted the viewfs-change branch May 18, 2017 21:37
vinishjail97 pushed a commit to vinishjail97/hudi that referenced this pull request Dec 15, 2023
…service fails (apache#7243) (apache#163)

After the files are written, table services like clustering and compaction can fail. This causes the sync to the metaserver to not happen. This patch adds a config that when set to false, the deltastreamer will not fail and the sync to the metaserver will occur. A warning will be logged with the exception that occurred. To use this new behavior, set hoodie.fail.writes.on.inline.table.service.exception to false.

Co-authored-by: Jonathan Vexler <=>

Co-authored-by: Jon Vexler <jbvexler@gmail.com>
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