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

HADOOP-10225 #137

Closed
wants to merge 1 commit into from

Conversation

@lewismc
Copy link

commented Oct 10, 2016

Hi Folks,
This PR is an attempt to address https://issues.apache.org/jira/browse/HADOOP-10225.
The patch can be tested by running

mvn release:clean release:prepare -DautoVersionSubmodules=true -DdryRun=true

Please let me know once it has been tested and I can squash commits into one.

@lewismc

This comment has been minimized.

Copy link
Author

commented Oct 10, 2016

@lewismc lewismc force-pushed the lewismc:HADOOP-10225 branch from 9f031f0 to 33c0b02 Oct 10, 2016
@lewismc

This comment has been minimized.

Copy link
Author

commented Oct 10, 2016

Just squashed all the commits so it is much cleaner.

@asfgit asfgit force-pushed the apache:trunk branch from 25ac447 to 4d1fac5 Apr 15, 2017
@elek

This comment has been minimized.

Copy link
Member

commented Jan 10, 2019

Can one of the admins verify this patch?

@lewismc

This comment has been minimized.

Copy link
Author

commented Jan 10, 2019

Wow this is a blast. I'm not sure how Hadoop is even released anymore. HADOOP-10225 is still open and unresolved. It might just be best to close this ticket... I am disappointed that no-one else on the committer team thought publishing javadoc and sources with the release was important... oh well.

@lewismc lewismc closed this Jan 10, 2019
@lewismc lewismc deleted the lewismc:HADOOP-10225 branch Jan 10, 2019
shanthoosh added a commit to shanthoosh/hadoop that referenced this pull request Oct 15, 2019
Author: Shanthoosh Venkataraman <svenkataraman@linkedin.com>

Reviewers: Xinyu Liu <xinyu@apache.org>

Closes apache#137 from shanthoosh/adding_logging_into_asyncrunloop
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.