[ 
https://issues.apache.org/jira/browse/HIVE-28178?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

László Bodor updated HIVE-28178:
--------------------------------
    Description: 
confusing things:
1. having a conf and jobConf: 
https://github.com/apache/hive/blob/77ca03509112655e81d3e56b71c938d548da8b7c/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/HiveSplitGenerator.java#L90-L91
 

2. inputInitializerContext != null check
https://github.com/apache/hive/blob/77ca03509112655e81d3e56b71c938d548da8b7c/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/HiveSplitGenerator.java#L261C13-L261C44
initializer context must be non-null, otherwise it's a serious bug in tez
we should not indent the whole code because of a confusing null check


> Cleanup some stuff in HiveSplitGenerator
> ----------------------------------------
>
>                 Key: HIVE-28178
>                 URL: https://issues.apache.org/jira/browse/HIVE-28178
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: László Bodor
>            Priority: Major
>
> confusing things:
> 1. having a conf and jobConf: 
> https://github.com/apache/hive/blob/77ca03509112655e81d3e56b71c938d548da8b7c/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/HiveSplitGenerator.java#L90-L91
>  
> 2. inputInitializerContext != null check
> https://github.com/apache/hive/blob/77ca03509112655e81d3e56b71c938d548da8b7c/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/HiveSplitGenerator.java#L261C13-L261C44
> initializer context must be non-null, otherwise it's a serious bug in tez
> we should not indent the whole code because of a confusing null check



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to