Skip to content

[TEZ-4439] Update the protobuf documentation in TEZ codebase - #233

Merged
abstractdog merged 1 commit into
apache:masterfrom
amanraj2520:TEZ-4439_UpdateDocumentation
Jul 26, 2022
Merged

[TEZ-4439] Update the protobuf documentation in TEZ codebase#233
abstractdog merged 1 commit into
apache:masterfrom
amanraj2520:TEZ-4439_UpdateDocumentation

Conversation

@amanraj2520

Copy link
Copy Markdown
Contributor

[TEZ-4439] Update the protobuf documentation in TEZ codebase.

Link to the issue : https://issues.apache.org/jira/browse/TEZ-4439

The protobuf dependency was upgraded from 2.5.0 to 3.19.4 as tracked in this issue : https://issues.apache.org/jira/browse/TEZ-4363. But there were certain mentions of protobuf 2.5.0 in the documentations of protobuf within the TEZ codebase. Raised this PR to update those documentations.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec32m 44sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+1 💚mvninstall16m 58smaster passed
+1 💚mvnsite0m 43smaster passed
_ Patch Compile Tests _
+1 💚mvninstall0m 10sthe patch passed
-1 ❌markdownlint0m 4sThe patch generated 1 new + 116 unchanged - 0 fixed = 117 total (was 116)
+1 💚mvnsite0m 9sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
_ Other Tests _
+1 💚asflicense0m 19sThe patch does not generate ASF License warnings.
51m 50s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-233/1/artifact/out/Dockerfile
GITHUB PR#233
JIRA IssueTEZ-4439
Optional Testsdupname asflicense markdownlint mvnsite
unameLinux 2275f681f3b0 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / c386865
markdownlinthttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-233/1/artifact/out/diff-patch-markdownlint.txt
Max. process+thread count77 (vs. ulimit of 5500)
modulesC: docs U: docs
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-233/1/console
versionsgit=2.25.1 maven=3.6.3 markdownlint=0.23.2
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@abstractdog

Copy link
Copy Markdown
Contributor

thanks @amanraj2520 , could you please check INSTALL.md (in root folder) for the same?

@amanraj2520

Copy link
Copy Markdown
ContributorAuthor

@abstractdog The INSTALL.md folder is symbolic linked to docs/src/site/markdown/install.md where I have made the changes.

@amanraj2520

Copy link
Copy Markdown
ContributorAuthor

@abstractdog Can you please let me know if you have any suggestions or are we good to merge this?

@abstractdog
abstractdog self-requested a review July 26, 2022 06:33
@abstractdog
abstractdog merged commit 5e31e4d into apache:masterJul 26, 2022
@abstractdog

Copy link
Copy Markdown
Contributor

merged to master, thanks a lot @amanraj2520 for taking care of docs

basapuram-kumar pushed a commit to acceldata-io/tez that referenced this pull request Sep 26, 2025
Sign up for freeto 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

@amanraj2520@tez-yetus@abstractdog