drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From arina-ielchiieva <...@git.apache.org>
Subject [GitHub] drill pull request #672: DRILL-5085: Add / update description for dynamic UD...
Date Tue, 06 Dec 2016 14:47:14 GMT
Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/672#discussion_r91085335
  
    --- 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 --
    
    Ok, reverted usage of generated temporary directory logic.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message