[jira] [Commented] (HIVE-11670) Strip out password information from TezSessionState configuration
[ https://issues.apache.org/jira/browse/HIVE-11670?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14975140#comment-14975140 ] Sushanth Sowmyan commented on HIVE-11670: - Note - while this patch was not committed in branch-1.2, in the process of backporting HIVE-9013, this was effectively merged in to the branch-1.2 commit for HIVE-9013 as well. > Strip out password information from TezSessionState configuration > - > > Key: HIVE-11670 > URL: https://issues.apache.org/jira/browse/HIVE-11670 > Project: Hive > Issue Type: Bug >Reporter: Hari Sankar Sivarama Subramaniyan >Assignee: Hari Sankar Sivarama Subramaniyan > Fix For: 1.3.0, 2.0.0 > > Attachments: HIVE-11670.1.patch > > > Remove password information from configuration copy that is sent to Yarn/Tez. > We don't need it there. The config entries can potentially be visible to > other users. > HIVE-10508 had the fix which removed this in certain places, however, when I > initiated a session via Hive Cli, I could still see the password information. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HIVE-11670) Strip out password information from TezSessionState configuration
[ https://issues.apache.org/jira/browse/HIVE-11670?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14717618#comment-14717618 ] Thejas M Nair commented on HIVE-11670: -- [~vikram.dixit] Can you please take a look. You know the tez code flow better. > Strip out password information from TezSessionState configuration > - > > Key: HIVE-11670 > URL: https://issues.apache.org/jira/browse/HIVE-11670 > Project: Hive > Issue Type: Bug >Reporter: Hari Sankar Sivarama Subramaniyan >Assignee: Hari Sankar Sivarama Subramaniyan > Attachments: HIVE-11670.1.patch > > > Remove password information from configuration copy that is sent to Yarn/Tez. > We don't need it there. The config entries can potentially be visible to > other users. > HIVE-10508 had the fix which removed this in certain places, however, when I > initiated a session via Hive Cli, I could still see the password information. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HIVE-11670) Strip out password information from TezSessionState configuration
[ https://issues.apache.org/jira/browse/HIVE-11670?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14717630#comment-14717630 ] Vikram Dixit K commented on HIVE-11670: --- LGTM +1. > Strip out password information from TezSessionState configuration > - > > Key: HIVE-11670 > URL: https://issues.apache.org/jira/browse/HIVE-11670 > Project: Hive > Issue Type: Bug >Reporter: Hari Sankar Sivarama Subramaniyan >Assignee: Hari Sankar Sivarama Subramaniyan > Attachments: HIVE-11670.1.patch > > > Remove password information from configuration copy that is sent to Yarn/Tez. > We don't need it there. The config entries can potentially be visible to > other users. > HIVE-10508 had the fix which removed this in certain places, however, when I > initiated a session via Hive Cli, I could still see the password information. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HIVE-11670) Strip out password information from TezSessionState configuration
[
https://issues.apache.org/jira/browse/HIVE-11670?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14720230#comment-14720230
]
Hive QA commented on HIVE-11670:
{color:red}Overall{color}: -1 at least one tests failed
Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12752863/HIVE-11670.1.patch
{color:red}ERROR:{color} -1 due to 3 failed/errored test(s), 9378 tests executed
*Failed tests:*
{noformat}
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver_index_auto_mult_tables_compact
org.apache.hive.hcatalog.api.TestHCatClient.testTableSchemaPropagation
org.apache.hive.hcatalog.hbase.TestPigHBaseStorageHandler.org.apache.hive.hcatalog.hbase.TestPigHBaseStorageHandler
{noformat}
Test results:
http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/5101/testReport
Console output:
http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/5101/console
Test logs:
http://ec2-174-129-184-35.compute-1.amazonaws.com/logs/PreCommit-HIVE-TRUNK-Build-5101/
Messages:
{noformat}
Executing org.apache.hive.ptest.execution.PrepPhase
Executing org.apache.hive.ptest.execution.ExecutionPhase
Executing org.apache.hive.ptest.execution.ReportingPhase
Tests exited with: TestsFailedException: 3 tests failed
{noformat}
This message is automatically generated.
ATTACHMENT ID: 12752863 - PreCommit-HIVE-TRUNK-Build
> Strip out password information from TezSessionState configuration
> -
>
> Key: HIVE-11670
> URL: https://issues.apache.org/jira/browse/HIVE-11670
> Project: Hive
> Issue Type: Bug
>Reporter: Hari Sankar Sivarama Subramaniyan
>Assignee: Hari Sankar Sivarama Subramaniyan
> Attachments: HIVE-11670.1.patch
>
>
> Remove password information from configuration copy that is sent to Yarn/Tez.
> We don't need it there. The config entries can potentially be visible to
> other users.
> HIVE-10508 had the fix which removed this in certain places, however, when I
> initiated a session via Hive Cli, I could still see the password information.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
[jira] [Commented] (HIVE-11670) Strip out password information from TezSessionState configuration
[ https://issues.apache.org/jira/browse/HIVE-11670?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14720918#comment-14720918 ] Hari Sankar Sivarama Subramaniyan commented on HIVE-11670: -- The failures are unrelated to the change. Thanks Hari > Strip out password information from TezSessionState configuration > - > > Key: HIVE-11670 > URL: https://issues.apache.org/jira/browse/HIVE-11670 > Project: Hive > Issue Type: Bug >Reporter: Hari Sankar Sivarama Subramaniyan >Assignee: Hari Sankar Sivarama Subramaniyan > Attachments: HIVE-11670.1.patch > > > Remove password information from configuration copy that is sent to Yarn/Tez. > We don't need it there. The config entries can potentially be visible to > other users. > HIVE-10508 had the fix which removed this in certain places, however, when I > initiated a session via Hive Cli, I could still see the password information. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
