hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jason Dere" <jd...@hortonworks.com>
Subject Re: Review Request 42286: HIVE-12853 LLAP: localize permanent UDF jars to daemon
Date Mon, 18 Jan 2016 18:42:04 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42286/#review114825
-----------------------------------------------------------



llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/FunctionLocalizer.java (line
63)
<https://reviews.apache.org/r/42286/#comment175671>

    You're initializing the classloader with the thread's current context class loader. Is
it possible the current class loader has loaded other JARs from some previous initialization
or query?



llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/FunctionLocalizer.java (line
65)
<https://reviews.apache.org/r/42286/#comment175894>

    I see you have created a work queue to handle localization of resources and updating the
classloader. Is there any mechanism in place to make sure nothing else is using the classloader
while it is being refreshed?


- Jason Dere


On Jan. 16, 2016, 12:09 a.m., Sergey Shelukhin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42286/
> -----------------------------------------------------------
> 
> (Updated Jan. 16, 2016, 12:09 a.m.)
> 
> 
> Review request for hive and Jason Dere.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> see jira
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 1e8c34b 
>   llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/ContainerRunnerImpl.java
0d85671 
>   llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/FunctionLocalizer.java
PRE-CREATION 
>   llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/LlapDaemon.java ddedfbf

>   llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/TaskExecutorService.java
34aa5c9 
>   llap-server/src/test/org/apache/hadoop/hive/llap/daemon/impl/TestTaskExecutorService.java
5491064 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionTask.java ec755a8 
>   ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java dca1198 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/LlapDecider.java af6129a 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java c38699d 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/ExprNodeGenericFuncDesc.java 25c2ff3 
>   ql/src/java/org/apache/hadoop/hive/ql/session/DependencyResolver.java 7289426 
>   ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 5f15557 
>   ql/src/java/org/apache/hadoop/hive/ql/util/DependencyResolver.java PRE-CREATION 
>   ql/src/java/org/apache/hadoop/hive/ql/util/DosToUnix.java 4480d54 
>   ql/src/java/org/apache/hadoop/hive/ql/util/ResourceDownloader.java PRE-CREATION 
>   ql/src/test/org/apache/hadoop/hive/ql/session/TestAddResource.java 1f07235 
> 
> Diff: https://reviews.apache.org/r/42286/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergey Shelukhin
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message