-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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
(3.3) Revert "HADOOP-16822. Provide source artifacts for hadoop-client-api" #6719
(3.3) Revert "HADOOP-16822. Provide source artifacts for hadoop-client-api" #6719
Conversation
… Contributed by Karel Kolman." (apache#6458) This reverts commit 2c4ab72. Justification: this was making debugging through IDEs worse, rather than better.
thanks; let's see what yetus says. +1 pending that |
@steveloughran One thing not related to this patch. I saw lots of Hadoop ecosystem projects using Yetus, but its comments are too too too long, actually, GitHub supports the collapsed format, which may improve the reviewer experience. |
@pan3793 interesting...though it's probably a yetus change. on long-lived patches I sometimes just delete the old ones, as really its only the last one or two which are of any relevance whatsoever |
💔 -1 overall
This message was automatically generated. |
thanks, merged |
This backport #6458 to branch-3.3
This reverts commit 2c4ab72.
Justification: this was making debugging through IDEs worse, rather than better.
Description of PR
How was this patch tested?
For code changes:
LICENSE
,LICENSE-binary
,NOTICE-binary
files?