drill-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DRILL-5085) Add / update description for dynamic UDFs directories in drill-env.sh and drill-module.conf
Date Fri, 02 Dec 2016 18:02:00 GMT

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

ASF GitHub Bot commented on DRILL-5085:
---------------------------------------

Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/672#discussion_r90688097
  
    --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionImplementationRegistry.java
---
    @@ -377,14 +374,12 @@ private ScanResult scan(ClassLoader classLoader, Path path, URL[]
urls) throws I
        * Creates local udf directory, if it doesn't exist.
        * Checks if local udf directory is a directory and if current application has write
rights on it.
        * Attempts to clean up local udf directory in case jars were left after previous drillbit
run.
    -   * Local udf directory path is concatenated from drill temporary directory and ${drill.exec.udf.directory.local}.
        *
        * @param config drill config
        * @return path to local udf directory
        */
       private Path getLocalUdfDir(DrillConfig config) {
    -    tmpDir = getTmpDir(config);
    -    File udfDir = new File(tmpDir, config.getString(ExecConstants.UDF_DIRECTORY_LOCAL));
    +    File udfDir = new File(config.getString(ExecConstants.UDF_DIRECTORY_LOCAL));
    --- End diff --
    
    Although we talked about using the new system you've implemented here, we have to consider
backward compatibility.
    
    Since the original behavior is already visible to users in Drill 1.8, I think we need
to leave your original design.
    
    At some point, we'll need to rationalize how Drill handles temp files and storage in DFS.
But, until then, your 1.8 design is fine.


> Add / update description for dynamic UDFs directories in drill-env.sh and drill-module.conf
> -------------------------------------------------------------------------------------------
>
>                 Key: DRILL-5085
>                 URL: https://issues.apache.org/jira/browse/DRILL-5085
>             Project: Apache Drill
>          Issue Type: Improvement
>    Affects Versions: 1.9.0
>            Reporter: Arina Ielchiieva
>            Assignee: Paul Rogers
>            Priority: Minor
>             Fix For: 1.10.0
>
>
> 1. Add description for $DRILL_TMP_DIR in drill-env.sh
> 2. Update description for dynamic UDFs directories in drill-module.conf



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message