Skip to content

Conversation

@piotte13
Copy link

@piotte13 piotte13 commented Jun 2, 2020

No description provided.

@hadoop-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 41s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 markdownlint 0m 1s markdownlint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ trunk Compile Tests _
+1 💚 mvninstall 24m 42s trunk passed
+1 💚 mvnsite 0m 45s trunk passed
+1 💚 shadedclient 41m 49s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 💚 mvninstall 0m 37s the patch passed
+1 💚 mvnsite 0m 38s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 shadedclient 16m 38s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 asflicense 0m 30s The patch does not generate ASF License warnings.
63m 45s
Subsystem Report/Notes
Docker ClientAPI=1.40 ServerAPI=1.40 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-2048/1/artifact/out/Dockerfile
GITHUB PR #2048
Optional Tests dupname asflicense mvnsite markdownlint
uname Linux d5a2d3548be3 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision trunk / 6288e15
Max. process+thread count 342 (vs. ulimit of 5500)
modules C: hadoop-tools/hadoop-aws U: hadoop-tools/hadoop-aws
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-2048/1/console
versions git=2.17.1 maven=3.6.0
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@aajisaka
Copy link
Member

aajisaka commented Jun 3, 2020

-1
The links are correct in the HTML file. https://hadoop.apache.org/docs/stable/hadoop-aws/tools/hadoop-aws/index.html
HTML documents are generated from the markdown files, and it is not supposed for users/developers to read the markdown files directly.

@piotte13
Copy link
Author

piotte13 commented Jun 3, 2020

Makes sense. Although, when I google for the aws documentation, the Md files are the first to appear, quite confusing.

We can close this PR then.

@piotte13 piotte13 closed this Jun 23, 2020
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