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

HDDS-1659. Define the process to add proposal/design docs to the Ozone subproject #922

Merged
merged 2 commits into from
Jun 11, 2019

Conversation

elek
Copy link
Member

@elek elek commented Jun 7, 2019

We think that it would be more effective to collect all the design docs in one place and make it easier to review them by the community.

We propose to follow an approach where the proposals are committed to the hadoop-hdds/docs project and the review can be the same as a review of a PR

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

@elek elek added the ozone label Jun 7, 2019

* Open a dedicated Jira (`HDDS-*` but with specific component)
* Use standard name prefix in the jira (easy to filter on the mailing list) `[OEP]
* Create a PR to merge the design doc (markdown) to `hadoop-hdds/docs/content/proposal` (will be part of the docs)
Copy link
Member Author

Choose a reason for hiding this comment

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

During an offline discussion @arp7 had some concerns about the usability of reviewing markdown files in PR vs. reviewing google docs.

Copy link
Member Author

Choose a reason for hiding this comment

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

Note: this is a complex PR because it contains the changes to show design docs on the docs page. A normal design doc would contain just one markdown file.

Copy link
Member Author

Choose a reason for hiding this comment

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

Update: I removed the HTTP/template changes.

@apache apache deleted a comment from hadoop-yetus Jun 7, 2019
@hadoop-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
0 reexec 106 Docker mode activated.
_ Prechecks _
+1 dupname 1 No case conflicting files found.
0 yamllint 1 yamllint was not available.
+1 @author 0 The patch does not contain any @author tags.
_ trunk Compile Tests _
+1 mvninstall 529 trunk passed
+1 mvnsite 0 trunk passed
+1 shadedclient 1364 branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 mvninstall 445 the patch passed
+1 mvnsite 0 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 730 patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 asflicense 51 The patch does not generate ASF License warnings.
2899
Subsystem Report/Notes
Docker Client=18.09.5 Server=18.09.5 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-922/2/artifact/out/Dockerfile
GITHUB PR #922
Optional Tests dupname asflicense mvnsite yamllint
uname Linux 658bedf076e5 4.15.0-48-generic #51-Ubuntu SMP Wed Apr 3 08:28:49 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision trunk / 8547957
Max. process+thread count 306 (vs. ulimit of 5500)
modules C: hadoop-hdds/docs U: hadoop-hdds/docs
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-922/2/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.

* [Apache Spark Project Improvement Proposals](https://spark.apache.org/improvement-proposals.html)
* [Kubernetes Enhancement Proposals](https://github.com/kubernetes/enhancements/tree/master/keps)

Short summary if the porcesses:
Copy link
Contributor

Choose a reason for hiding this comment

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

typo

@ajayydv
Copy link
Contributor

ajayydv commented Jun 10, 2019

i like the idea of more detailed information in markdown format. It will be easier to update in subsequent changes and will be included in docs in case some user wants to access it.

@anuengineer
Copy link
Contributor

At all, this is just the start of this conversation, let us commit this and iterate over this process and get into a good shape. Thank you all for a wonderful discussion during the community call.

@anuengineer anuengineer merged commit e997f2a into apache:trunk Jun 11, 2019
@elek
Copy link
Member Author

elek commented Jun 12, 2019

Thanks to commit this @anuengineer I repoened it temporary as it introduced a RAT error in daily build.

Will upload a fixed version soon.

elek added a commit that referenced this pull request Jun 12, 2019
bshashikant pushed a commit to bshashikant/hadoop that referenced this pull request Jul 10, 2019
…e subproject (apache#922)

* HDDS-1659. Define the process to add proposal/design docs to the Ozone subproject

* Remove Site improvements to display proposals
bshashikant pushed a commit to bshashikant/hadoop that referenced this pull request Jul 10, 2019
shanthoosh pushed a commit to shanthoosh/hadoop that referenced this pull request Oct 15, 2019
Checkstyle version upgrade 5.9 -> 6.11 and fixes to pass checks

**Why this version (6.11) but not latest (8.17)?**
In version [6.11.1](http://checkstyle.sourceforge.net/releasenotes.html#Release_6.11.1) lambda indentation support was added, and it reports tremendous amount of errors in project. I found no option to change indent policy for lambdas therefore set version without this check.

Author: strkkk <andreypaykin@gmail.com>

Reviewers: Sanil Jain <snjain@linkedin.com>

Closes apache#922 from strkkk/checkstyle
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