Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

R SDK For ADLS: Rename all APIs similar to rhdfs #15

Merged
merged 1 commit into from Jul 6, 2018

Conversation

omkarksa
Copy link
Owner

@omkarksa omkarksa commented Jul 5, 2018

export(adls.fileinputstream.close)
export(adls.fileinputstream.getpos)
export(adls.fileinputstream.length)
export(adls.fileinputstream.mark)

Choose a reason for hiding this comment

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

AFAIK these are just there from Java InputStream definition, so the mark methods can be removed from R.

Copy link
Owner Author

Choose a reason for hiding this comment

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

trying to make it similar to the ADLS Java SDK. Here too the execution stops with a message like "UnsupportedOperationException: mark()/reset() not supported on this stream ..."

export(adlFileOutputStreamFlush)
export(adlFileOutputStreamWrite)
export(adls.append)
export(adls.append.direct)

Choose a reason for hiding this comment

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

What does append.direct do as opposed to just append? (same for read and read.direct)

Copy link
Owner Author

@omkarksa omkarksa Jul 6, 2018

Choose a reason for hiding this comment

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

The .direct methods complete the operation in one step, without providing the user with a file IO streams.

@omkarksa omkarksa merged commit 1900063 into master Jul 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants