Skip to content

Conversation

@elek
Copy link
Member

@elek elek commented Jul 4, 2019

In HDDS-1639 we restructured the ozone documentation and a new overview page is added to the main page.

This page contains an official aws logo, As [~bharatviswa] reported we are not sure about the exact condition to use logos / trademarks from Amazon. It's better to remain on the safe side and use a neutral S3 label.

In this patch the aws logo is replaced with an orange cloud + s3 text.

See: https://issues.apache.org/jira/browse/HDDS-1763

@elek elek added the ozone label Jul 4, 2019
@hadoop-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
0 reexec 33 Docker mode activated.
_ Prechecks _
+1 dupname 1 No case conflicting files found.
+1 @author 0 The patch does not contain any @author tags.
_ trunk Compile Tests _
+1 shadedclient 760 branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 658 patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 asflicense 32 The patch does not generate ASF License warnings.
1603
Subsystem Report/Notes
Docker Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1058/1/artifact/out/Dockerfile
GITHUB PR #1058
Optional Tests dupname asflicense
uname Linux f45758bdf4d9 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision trunk / 1c254a8
Max. process+thread count 401 (vs. ulimit of 5500)
modules C: hadoop-hdds/docs U: hadoop-hdds/docs
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-1058/1/console
versions git=2.7.4 maven=3.3.9
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

Copy link
Contributor

@mukul1987 mukul1987 left a comment

Choose a reason for hiding this comment

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

+1, the change looks good to me.

@anuengineer anuengineer merged commit 47345e5 into apache:trunk Jul 15, 2019
asfgit pushed a commit that referenced this pull request Jul 15, 2019
HDDS-1763. Use vendor neutral s3 logo in ozone doc

(cherry picked from commit 47345e5)
shanthoosh pushed a commit to shanthoosh/hadoop that referenced this pull request Oct 15, 2019
Returning the AtomicInteger instances directly for `processorCount` was causing issues with our InfluxDB reporter. I added the `.get()` to match all the others.

Author: thunderstumpges <[email protected]>

Reviewers: Prateek Maheshwari <[email protected]>

Closes apache#1058 from thunderstumpges/atomic-int-metrics-fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants