[
https://issues.apache.org/jira/browse/HADOOP-17482?focusedWorklogId=543103&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-543103
]
ASF GitHub Bot logged work on HADOOP-17482:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 27/Jan/21 21:36
Start Date: 27/Jan/21 21:36
Worklog Time Spent: 10m
Work Description: goiri commented on a change in pull request #2633:
URL: https://github.com/apache/hadoop/pull/2633#discussion_r565650951
##########
File path:
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FileSystem.java
##########
@@ -3391,15 +3391,7 @@ private static void loadFileSystems() {
LOGGER.info("Full exception loading: {}", fs, e);
}
} catch (ServiceConfigurationError ee) {
- LOG.warn("Cannot load filesystem: " + ee);
- Throwable cause = ee.getCause();
- // print all the nested exception messages
- while (cause != null) {
- LOG.warn(cause.toString());
- cause = cause.getCause();
- }
- // and at debug: the full stack
- LOG.debug("Stack Trace", ee);
+ LOGGER.warn("Cannot load filesystem", ee);
Review comment:
I think relying on slf4j and just dumping the whole exception is good.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 543103)
Time Spent: 3h 40m (was: 3.5h)
> Remove Commons Logger from FileSystem Class
> -------------------------------------------
>
> Key: HADOOP-17482
> URL: https://issues.apache.org/jira/browse/HADOOP-17482
> Project: Hadoop Common
> Issue Type: Improvement
> Components: common
> Reporter: David Mollitor
> Assignee: David Mollitor
> Priority: Minor
> Labels: pull-request-available
> Time Spent: 3h 40m
> Remaining Estimate: 0h
>
> Remove reference to Commons Logger in FileSystem, it already has SLF4J, so
> it's a bit weird to be mixing and matching and interweaving loggers in this
> way. Also, my hope is to eventually migrate everything to SLF4J to simplify
> things for downstream consumers of the common library.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
