Skip to content

Conversation

@vinayakumarb
Copy link
Contributor

CHANGES

Upgrades the hadoop-thirdparty shaded-protobuf_3.7 artifact to 1.0.0 released version

Copy link
Member

@ayushtkn ayushtkn left a comment

Choose a reason for hiding this comment

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

+1
Give a check, if Jenkins results, may be it would be timeout, since the changes are at project root...
Thanx @vinayakumarb for the update..

@vinayakumarb
Copy link
Contributor Author

+1
Give a check, if Jenkins results, may be it would be timeout, since the changes are at project root...
Thanx @vinayakumarb for the update..

Thanks @ayushtkn .
Here is the run https://builds.apache.org/view/H-L/view/Hadoop/job/hadoop-multibranch/view/change-requests/job/PR-1900/1/console
Not sure why comment didnt come here.

@ayushtkn
Copy link
Member

Thanx @vinayakumarb for checking. Seems comment isn't coming, some problem, will check. it isn't coming for patches too.

Anyway the build seems cleans.

@asfgit asfgit merged commit f02d5ab into apache:trunk Mar 20, 2020
@vinayakumarb
Copy link
Contributor Author

Thanks @ayushtkn for review. Merged to trunk.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 0s Docker mode activated.
-1 ❌ patch 0m 4s #1900 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help.
Subsystem Report/Notes
GITHUB PR #1900
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-1900/2/console
versions git=2.17.1
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

bilaharith pushed a commit to bilaharith/hadoop that referenced this pull request Mar 23, 2020
RogPodge pushed a commit to RogPodge/hadoop that referenced this pull request Mar 25, 2020
zhangxiping1 pushed a commit to zhangxiping1/hadoop that referenced this pull request Dec 13, 2022
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.

4 participants