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

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

commit 42fda24364786cc1a457890bd212bb3922479e95
Author: Michael Smith <[email protected]>
AuthorDate: Tue Oct 15 10:31:23 2024 -0700

    IMPALA-13452: Add test scope for dependencies
    
    Adds the test scope to dependencies derby, javax.ws.rs-api, javax.json
    so they're not included in packaged artifacts, as they're only needed
    for testing. This reduces the security surface area and artifact size.
    
    Also removes javax.mail, as it's no longer needed for
    ranger-plugins-audit.
    
    Change-Id: Iba170c190df47389e08a9a4325aeb1e7ca0bd98d
    Reviewed-on: http://gerrit.cloudera.org:8080/21933
    Tested-by: Impala Public Jenkins <[email protected]>
    Reviewed-by: Andrew Sherman <[email protected]>
---
 fe/pom.xml | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/fe/pom.xml b/fe/pom.xml
index 1275a5a7a..529dafa7b 100644
--- a/fe/pom.xml
+++ b/fe/pom.xml
@@ -224,17 +224,11 @@ under the License.
       </exclusions>
     </dependency>
 
-    <!-- This is needed by ranger-plugins-audit -->
-    <dependency>
-      <groupId>javax.mail</groupId>
-      <artifactId>mail</artifactId>
-      <version>1.4</version>
-    </dependency>
-
     <dependency>
       <groupId>javax.ws.rs</groupId>
       <artifactId>javax.ws.rs-api</artifactId>
       <version>2.1.1</version>
+      <scope>test</scope>
     </dependency>
 
     <dependency>
@@ -338,6 +332,7 @@ under the License.
       <groupId>org.apache.derby</groupId>
       <artifactId>derby</artifactId>
       <version>${derby.version}</version>
+      <scope>test</scope>
     </dependency>
 
     <dependency>
@@ -381,6 +376,7 @@ under the License.
       <groupId>org.glassfish</groupId>
       <artifactId>javax.json</artifactId>
       <version>1.0.2</version>
+      <scope>test</scope>
     </dependency>
 
     <dependency>

Reply via email to