[jira] [Commented] (HIVE-5702) add logging on WebHCat startup to print all env var and Configuration

["Hive QA (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/HIVE-5702?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13854839#comment-13854839 ] 

Hive QA commented on HIVE-5702:
-------------------------------



{color:green}Overall{color}: +1 all checks pass

Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12619910/HIVE-5702.patch

{color:green}SUCCESS:{color} +1 4811 tests passed

Test results: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/722/testReport
Console output: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/722/console

Messages:
{noformat}
Executing org.apache.hive.ptest.execution.PrepPhase
Executing org.apache.hive.ptest.execution.ExecutionPhase
Executing org.apache.hive.ptest.execution.ReportingPhase
{noformat}

This message is automatically generated.

ATTACHMENT ID: 12619910

> add logging on WebHCat startup to print all env var and Configuration
> ---------------------------------------------------------------------
>
>                 Key: HIVE-5702
>                 URL: https://issues.apache.org/jira/browse/HIVE-5702
>             Project: Hive
>          Issue Type: Bug
>          Components: WebHCat
>    Affects Versions: 0.13.0
>            Reporter: Eugene Koifman
>            Assignee: Eugene Koifman
>            Priority: Minor
>         Attachments: HIVE-5702.patch
>
>
> this helps debugging/diagnosing issues
> Something like:
> Map<String, String> env = System.getenv();
>       List<String> propKeys = new ArrayList<String>(env.keySet());
>       Collections.sort(propKeys);
>       LOG.info("WebHCat environment:");
>       for(String propKey : propKeys) {
>         LOG.info(propKey + "=" + env.get(propKey));
>       }
>       LOG.info("Configuration props:");
>       Iterator<Map.Entry<String, String>> configProps = appConf.iterator();
>       while(configProps.hasNext()) {
>         Map.Entry<String, String> ent = configProps.next();
>         LOG.info(ent.getKey() + "=" + ent.getValue());
>       }
> Could be useful to add a REST call to dump such info as well  (is security a concern here?)
> Also, make sure to log what config files it attempted to load and which it actually did load.
>   



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)