Skip to content

HADOOP-17514. Remove trace subcommand from hadoop CLI. #2680

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

Merged
merged 1 commit into from
Mar 12, 2021

Conversation

iwasakims
Copy link
Member

https://issues.apache.org/jira/browse/HADOOP-17514

TraceAdmin protocol and utility class were removed by HADOOP-17424.

@hadoop-yetus

This comment has been minimized.

@hadoop-yetus

This comment has been minimized.

Copy link
Contributor

@jojochuang jojochuang left a comment

Choose a reason for hiding this comment

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

Assuming we will ultimately switch to OpenTracing/OpenTelemetry, we still want a trace subcommand?

@iwasakims
Copy link
Member Author

iwasakims commented Feb 23, 2021

@jojochuang TraceAdmin protocol was removed by @smengcl because it is not needed for alternative tracer impl.
#2645 (comment)

If it turned out to be needed, we can add it back again. Since trace subcommand was actually dynamic reconfiguration functionality, adding generic protocol which can be used for other purposes could be an option.

@smengcl
Copy link
Contributor

smengcl commented Mar 11, 2021

Good catch. Actually because TraceAdmin is removed, the trace subcommand is referencing a non-existent class now:

$ ./bin/hadoop trace
Error: Could not find or load main class org.apache.hadoop.tracing.TraceAdmin

I'm +1 on removing this right now.

@iwasakims
Copy link
Member Author

Thanks, @smengcl. I'm merging this.

@iwasakims iwasakims merged commit 0f6ba5e into apache:trunk Mar 12, 2021
kiran-maturi pushed a commit to kiran-maturi/hadoop that referenced this pull request Nov 24, 2021
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