This is an automated email from the ASF dual-hosted git repository.

abhay pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ranger.git

commit 003ac834be8990b7422c14ac623e76a1eeb7fb2f
Author: Abhay Kulkarni <[email protected]>
AuthorDate: Mon Jul 8 13:33:52 2019 -0700

    Revert "RANGER-2489: Missing dependencies in assembly for Presto plugin"
    
    This reverts commit 02251177c90bf023fc50729b2a20e90f9ab52101.
    
    The author of the patch is incorrect.
---
 agents-common/scripts/enable-agent.sh | 11 +++--------
 ranger-presto-plugin-shim/pom.xml     | 24 ------------------------
 src/main/assembly/plugin-presto.xml   |  5 -----
 3 files changed, 3 insertions(+), 37 deletions(-)

diff --git a/agents-common/scripts/enable-agent.sh 
b/agents-common/scripts/enable-agent.sh
index badea18..7dec0b1 100755
--- a/agents-common/scripts/enable-agent.sh
+++ b/agents-common/scripts/enable-agent.sh
@@ -213,7 +213,7 @@ elif [ "${HCOMPONENT_NAME}" = "kylin" ]; then
 elif [ "${HCOMPONENT_NAME}" = "elasticsearch" ]; then
     HCOMPONENT_LIB_DIR=${HCOMPONENT_INSTALL_DIR}/plugins
 elif [ "${HCOMPONENT_NAME}" = "presto" ]; then
-    HCOMPONENT_LIB_DIR=${HCOMPONENT_INSTALL_DIR}/plugin/ranger
+    HCOMPONENT_LIB_DIR=${HCOMPONENT_INSTALL_DIR}/plugins/ranger
     if [ ! -d "${HCOMPONENT_LIB_DIR}" ]; then
         echo "INFO: Creating ${HCOMPONENT_LIB_DIR}"
         mkdir -p ${HCOMPONENT_LIB_DIR}
@@ -798,14 +798,9 @@ then
                bf=${dn}/.${bn}.${dt}
                echo "backup of ${fn} to ${bf} ..."
                cp ${fn} ${bf}
-       else
-           fn=${HCOMPONENT_CONF_DIR}/access-control.properties
+               echo "Add or Update properties file: [${fn}] ... "
+               addOrUpdatePropertyToFile access-control.name $controlName ${fn}
        fi
-       echo "Add or Update properties file: [${fn}] ... "
-       addOrUpdatePropertyToFile access-control.name $controlName ${fn}
-       echo "Linking config files"
-       cd ${HCOMPONENT_LIB_DIR}/ranger-presto-plugin-impl/
-       ln -sf ${HCOMPONENT_CONF_DIR} conf
 fi
 
 
diff --git a/ranger-presto-plugin-shim/pom.xml 
b/ranger-presto-plugin-shim/pom.xml
index d5338fc..91ca8e8 100644
--- a/ranger-presto-plugin-shim/pom.xml
+++ b/ranger-presto-plugin-shim/pom.xml
@@ -101,29 +101,5 @@
             <version>${zookeeper.version}</version>
         </dependency>
 
-        <dependency>
-            <groupId>net.java.dev.jna</groupId>
-            <artifactId>jna</artifactId>
-            <version>${jna.version}</version>
-        </dependency>
-
-        <dependency>
-            <groupId>net.java.dev.jna</groupId>
-            <artifactId>jna-platform</artifactId>
-            <version>${jna-platform.version}</version>
-        </dependency>
-
-        <dependency>
-            <groupId>commons-codec</groupId>
-            <artifactId>commons-codec</artifactId>
-            <version>${commons.codec.version}</version>
-        </dependency>
-
-        <dependency>
-            <groupId>com.kstruct</groupId>
-            <artifactId>gethostname4j</artifactId>
-            <version>${kstruct.gethostname4j.version}</version>
-        </dependency>
-
     </dependencies>
 </project>
\ No newline at end of file
diff --git a/src/main/assembly/plugin-presto.xml 
b/src/main/assembly/plugin-presto.xml
index 17260ab..8063a9d 100644
--- a/src/main/assembly/plugin-presto.xml
+++ b/src/main/assembly/plugin-presto.xml
@@ -80,7 +80,6 @@
                             <include>commons-cli:commons-cli</include>
                             
<include>commons-collections:commons-collections</include>
                             
<include>org.apache.commons:commons-configuration2:jar:${commons.configuration.version}</include>
-                            <include>commons-codec:commons-codec</include>
                             
<include>commons-io:commons-io:jar:${commons.io.version}</include>
                             
<include>commons-lang:commons-lang:jar:${commons.lang.version}</include>
                             
<include>commons-logging:commons-logging:jar:${commons.logging.version}</include>
@@ -99,9 +98,6 @@
                             
<include>org.codehaus.jackson:jackson-mapper-asl</include>
                             <include>org.codehaus.jackson:jackson-xc</include>
                             
<include>org.apache.zookeeper:zookeeper:jar:${zookeeper.version}</include>
-                            <include>net.java.dev.jna:jna</include>
-                            <include>net.java.dev.jna:jna-platform</include>
-                            <include>com.kstruct:gethostname4j</include>
                         </includes>
                     </dependencySet>
                 </dependencySets>
@@ -124,7 +120,6 @@
                         <includes>
                             <include>commons-cli:commons-cli</include>
                             
<include>commons-collections:commons-collections</include>
-                            
<include>com.google.guava:guava:jar:${presto.guava.version}</include>
                             
<include>org.apache.commons:commons-configuration2:jar:${commons.configuration.version}</include>
                             
<include>commons-io:commons-io:jar:${commons.io.version}</include>
                             <include>commons-lang:commons-lang</include>

Reply via email to