[
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)