----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72541/ -----------------------------------------------------------
Review request for ranger, Ankita Sinha, Bolke de Bruin, Don Bosco Durai, bhavik patel, Colm O hEigeartaigh, Gautam Borad, Jayendra Parab, Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Nikhil P, Nitin Galave, pengjianhua, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, Velmurugan Periasamy, Qiang Zhang, and Zsombor Gegesy. Bugs: RANGER-2831 https://issues.apache.org/jira/browse/RANGER-2831 Repository: ranger Description ------- **Problem Statement:** After RANGER-2826 commit presto-server-333 start failed with below error. ERROR main io.prestosql.server.PrestoServer javax/annotation/PostConstructjava.lang.NoClassDefFoundError: javax/annotation/PostConstruct at io.airlift.bootstrap.LifeCycleMethods.addLifeCycleMethods(LifeCycleMethods.java:65) at io.airlift.bootstrap.LifeCycleMethods.<init>(LifeCycleMethods.java:39) at java.base/java.util.concurrent.ConcurrentHashMap.computeIfAbsent(ConcurrentHashMap.java:1705) at io.airlift.bootstrap.LifeCycleMethodsMap.get(LifeCycleMethodsMap.java:27) at io.airlift.bootstrap.LifeCycleModule.isLifeCycleClass(LifeCycleModule.java:86) at io.airlift.bootstrap.LifeCycleModule.provision(LifeCycleModule.java:55) at **Proposed solution:** javax.annotation-api library need to be added in ranger-2.1.0-SNAPSHOT-presto-plugin/lib location. **Note:** Proposed patch will also add the httpcore and zookeeper libraries at ranger-2.1.0-SNAPSHOT-presto-plugin/lib/ranger-presto-plugin-impl location as reported in by "Jie Zhang" in RANGER-2826 (https://issues.apache.org/jira/browse/RANGER-2831?focusedCommentId=17113580&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-17113580) Diffs ----- distro/src/main/assembly/plugin-presto.xml 507944b53 plugin-presto/pom.xml 61728b543 pom.xml 0b2e09263 ranger-presto-plugin-shim/pom.xml 5c0f897ee security-admin/src/main/java/org/apache/ranger/patch/PatchForPrestoToSupportPresto333_J10038.java f47ad1309 Diff: https://reviews.apache.org/r/72541/diff/1/ Testing ------- Tested the build and ranger admin installation. Thanks, Pradeep Agrawal