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

Sergey Shelukhin commented on HIVE-17431:
-----------------------------------------

[~sseth] can you please take a look? https://reviews.apache.org/r/62048/ 
Just to make sure there isn't some hidden behavior that I'm missing. The tests 
appear to pass.

> change configuration handling in TezSessionState
> ------------------------------------------------
>
>                 Key: HIVE-17431
>                 URL: https://issues.apache.org/jira/browse/HIVE-17431
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Sergey Shelukhin
>            Assignee: Sergey Shelukhin
>         Attachments: HIVE-17431.patch
>
>
> The configuration is only set when opening the session; that seems 
> unnecessary. E.g. when updating the session and localizing new resources we 
> may theoretically open the session with a new config, but we don't update the 
> config and only update the files if the session is already open, which seems 
> to imply that it's ok to not update the config. 
> In most cases, the session is opened only once or reopened without intending 
> to change the config (e.g. if it times out).



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to