-
Notifications
You must be signed in to change notification settings - Fork 9.2k
HADOOP-13655 document object store use with fs shell and distcp #131
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
HADOOP-13655 document object store use with fs shell and distcp #131
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
crc should be -crc
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hadoop fs -copyFromLocal -d -f /datasets/devices.orc s3a://bucket/datasets/" is redundant, right?
The symbol "
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hadoop fs -put -d -f - wasb: should be hadoop fs -put -d -f wasb:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
actually no! I couldn't get the put command to read off stdin according to the (Existing) docs, so I looked in the source code and saw that you needed a special "-" as the source, otherwise the s3a url :// was being treated as a source and the command failing as the destination was missing. It does make sense —and is consistent with other tools, just not documented completely.
I'd mentioned this in the "put" section, but I've expanded the comment here and added the - to the DNF form of the put command.
liuml07
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
This is excellent doc; we should get this in ASAP.
This is for branch-2? How about trunk?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
objec -> object
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<a name="ObjectStores" /> is accidently here I guess?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, it's to have a link which can be referred to later
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The -diff/-rdiff option is not supported
Yes there is an rdiff options that is just added.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The indention is unnecessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it is needed to indent the paragraph and code sample under the bullet point.
Change-Id: I0f50d0a7823a86a9e0b1fdfc70a69fcebd1d22ce
0fea00c to
881366f
Compare
Join TTL was set too low (10 ms). `joinRetainsMatchedMessagesReverse` will fail if the execution time between line 176 and 186 is longer than that. Increased TTL to 1 min (will not affect test execution time). Author: Prateek Maheshwari <[email protected]> Reviewers: Navina Ramesh <[email protected]> Closes apache#131 from prateekm/join-test-fix
Patch of filesystem shell & distcp docs to cover object stores. Also updated some references in filesystem/index.md which were out of date