hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sergey Shelukhin (JIRA)" <>
Subject [jira] [Commented] (HIVE-12237) Use slf4j as logging facade
Date Thu, 29 Oct 2015 00:08:27 GMT


Sergey Shelukhin commented on HIVE-12237:

I think there may be many such lines in the codebase and no test will reliably find them all.
We should at least have code review of script that replaces these and make sure methods are
not replaced.
Overall I don't see why there's rush to remove the checks. This can be handled on case by
case basis. We should revert, and commit only the non-automated parts.
Then if we do want to remove the checks, do the above.

> Use slf4j as logging facade
> ---------------------------
>                 Key: HIVE-12237
>                 URL:
>             Project: Hive
>          Issue Type: Task
>          Components: Logging
>    Affects Versions: 2.0.0
>            Reporter: Ashutosh Chauhan
>            Assignee: Ashutosh Chauhan
>             Fix For: 2.0.0
>         Attachments: HIVE-12237.1.patch, HIVE-12237.2.patch, HIVE-12237.3.patch, HIVE-12237.4.patch,
HIVE-12237.5.patch, HIVE-12237.6.patch, HIVE-12237.patch

This message was sent by Atlassian JIRA

View raw message