[jira] [Comment Edited] (HIVE-23294) Remove sync bottleneck in TezConfigurationFactory

["Ashutosh Chauhan (Jira)" <ji...@apache.org>]

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

Ashutosh Chauhan edited comment on HIVE-23294 at 4/26/20, 6:33 AM:
-------------------------------------------------------------------

[~rajesh.balamohan] what do you think of my patch. Arguably it might be faster since we never hit synchronized method nor we need to worry about memory. Though reflection access might not be as fast as hashmap lookup.


was (Author: ashutoshc):
[~rajesh.balamohan] what do you think of my patch. Arguably it might be faster since we never hit synchronized method nor we need to worry about memory. Though reflection access might be as fast as hashmap lookup.

> Remove sync bottleneck in TezConfigurationFactory
> -------------------------------------------------
>
>                 Key: HIVE-23294
>                 URL: https://issues.apache.org/jira/browse/HIVE-23294
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Rajesh Balamohan
>            Assignee: Rajesh Balamohan
>            Priority: Major
>         Attachments: HIVE-23294.1.patch, HIVE-23294.2.patch, Screenshot 2020-04-24 at 1.53.20 PM.png
>
>
> [https://github.com/apache/hive/blob/master/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezConfigurationFactory.java#L53]
> [https://github.com/apache/hadoop/blob/master/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/conf/Configuration.java#L1628]
> It ends up locking for property names in the config. For short running queries with concurrency, this is an issue.
>  
> !Screenshot 2020-04-24 at 1.53.20 PM.png|width=1086,height=459!
>  
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)