You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hive.apache.org by "Gergely Hajós (JIRA)" <ji...@apache.org> on 2018/03/02 10:18:00 UTC
[jira] [Commented] (HIVE-18716) Delete unnecessary parameters from
TaskFactory
[ https://issues.apache.org/jira/browse/HIVE-18716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16383426#comment-16383426 ]
Gergely Hajós commented on HIVE-18716:
--------------------------------------
[~kgyrtkirk] Maybe I'm wrong, but I think that I already fixed \{code} createAuthorizationTaskFactory(HiveConf conf, Hive db) \{code} in patch #3. Please tell me if I'm missing something!
> Delete unnecessary parameters from TaskFactory
> ----------------------------------------------
>
> Key: HIVE-18716
> URL: https://issues.apache.org/jira/browse/HIVE-18716
> Project: Hive
> Issue Type: Improvement
> Components: HiveServer2
> Affects Versions: 3.0.0
> Reporter: Gergely Hajós
> Assignee: Gergely Hajós
> Priority: Trivial
> Attachments: HIVE-18716.1.patch, HIVE-18716.2.patch, HIVE-18716.3.patch
>
>
> * In _TaskFactory class conf_ parameter is not used here
> {code:java}
> public static <T extends Serializable> Task<T> get(Class<T> workClass,
> HiveConf conf)
> {code}
> * _get_ functions never called with vararg parameter _Task<? extends Serializable>... tasklist_
> * the following version of _get_ is always called with _setConf_ true from outside of _TaskFactory_ class
> {code:java}
> public static <T extends Serializable> Task<T> get(T work, HiveConf conf, boolean setConf, Task<? extends Serializable>... tasklist)
> {code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)