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

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


The following commit(s) were added to refs/heads/master by this push:
     new c7e070a  Removing Nullable annotation and fix dependency errors
c7e070a is described below

commit c7e070a97c1f92b1d0bc527e12d07b3bcee04dbf
Author: Martin Stockhammer <[email protected]>
AuthorDate: Sun Feb 2 16:31:30 2020 +0100

    Removing Nullable annotation and fix dependency errors
    
    Nullable annotation is not part of javax.annotation-api package. To avoid 
having the
    additional dependency to findbugs ( no current updates ) I removed the 
Nullable
    annotations until we find a better approach.
---
 .../archiva-maven/archiva-maven-repository/pom.xml     |  6 ++++++
 .../archiva-maven/archiva-maven-scheduler/pom.xml      |  5 +++++
 .../archiva-scheduler-repository/pom.xml               |  5 +++++
 .../metadata/repository/MetadataRepository.java        | 18 +++++++-----------
 .../cassandra/CassandraMetadataRepository.java         |  2 --
 .../repository/file/FileMetadataRepository.java        |  2 --
 .../metadata/repository/jcr/JcrMetadataRepository.java |  2 --
 7 files changed, 23 insertions(+), 17 deletions(-)

diff --git a/archiva-modules/archiva-maven/archiva-maven-repository/pom.xml 
b/archiva-modules/archiva-maven/archiva-maven-repository/pom.xml
index 94d7e7f..8fb5a38 100644
--- a/archiva-modules/archiva-maven/archiva-maven-repository/pom.xml
+++ b/archiva-modules/archiva-maven/archiva-maven-repository/pom.xml
@@ -152,6 +152,12 @@
 
     <!-- Test scope -->
     <dependency>
+      <groupId>org.apache.maven.indexer</groupId>
+      <artifactId>indexer-core</artifactId>
+      <classifier>shaded-lucene</classifier>
+      <scope>test</scope>
+    </dependency>
+    <dependency>
       <groupId>org.xmlunit</groupId>
       <artifactId>xmlunit-core</artifactId>
       <scope>test</scope>
diff --git a/archiva-modules/archiva-maven/archiva-maven-scheduler/pom.xml 
b/archiva-modules/archiva-maven/archiva-maven-scheduler/pom.xml
index 9254ede..b24adb4 100644
--- a/archiva-modules/archiva-maven/archiva-maven-scheduler/pom.xml
+++ b/archiva-modules/archiva-maven/archiva-maven-scheduler/pom.xml
@@ -62,6 +62,11 @@
     </dependency>
 
     <dependency>
+      <groupId>org.apache.maven.indexer</groupId>
+      <artifactId>indexer-core</artifactId>
+      <classifier>shaded-lucene</classifier>
+    </dependency>
+    <dependency>
       <groupId>org.apache.maven.wagon</groupId>
       <artifactId>wagon-http</artifactId>
       <scope>provided</scope>
diff --git 
a/archiva-modules/archiva-scheduler/archiva-scheduler-repository/pom.xml 
b/archiva-modules/archiva-scheduler/archiva-scheduler-repository/pom.xml
index 46a01e4..a312519 100644
--- a/archiva-modules/archiva-scheduler/archiva-scheduler-repository/pom.xml
+++ b/archiva-modules/archiva-scheduler/archiva-scheduler-repository/pom.xml
@@ -88,6 +88,11 @@
 
     <!-- Test scope -->
     <dependency>
+      <groupId>org.apache.maven.indexer</groupId>
+      <artifactId>indexer-core</artifactId>
+      <classifier>shaded-lucene</classifier>
+    </dependency>
+    <dependency>
       <groupId>commons-beanutils</groupId>
       <artifactId>commons-beanutils</artifactId>
       <scope>test</scope>
diff --git 
a/archiva-modules/metadata/metadata-repository-api/src/main/java/org/apache/archiva/metadata/repository/MetadataRepository.java
 
b/archiva-modules/metadata/metadata-repository-api/src/main/java/org/apache/archiva/metadata/repository/MetadataRepository.java
index c63420a..abaed59 100644
--- 
a/archiva-modules/metadata/metadata-repository-api/src/main/java/org/apache/archiva/metadata/repository/MetadataRepository.java
+++ 
b/archiva-modules/metadata/metadata-repository-api/src/main/java/org/apache/archiva/metadata/repository/MetadataRepository.java
@@ -26,8 +26,6 @@ import org.apache.archiva.metadata.model.ProjectMetadata;
 import org.apache.archiva.metadata.model.ProjectVersionMetadata;
 import org.apache.archiva.metadata.model.ProjectVersionReference;
 
-import javax.annotation.Nullable;
-import javax.annotation.ParametersAreNonnullByDefault;
 import java.time.ZonedDateTime;
 import java.util.List;
 import java.util.stream.Stream;
@@ -92,8 +90,6 @@ import java.util.stream.Stream;
  * lazy loading for the retrieved objects. So the streaming methods may be 
faster and use less memory than the list based methods.
  * But for some backends there is no difference.
  */
-@SuppressWarnings( "NullableProblems" )
-@ParametersAreNonnullByDefault
 public interface MetadataRepository
 {
 
@@ -289,7 +285,7 @@ public interface MetadataRepository
      * uses default query parameters.
      */
     List<ArtifactMetadata> getArtifactsByDateRange( RepositorySession session, 
String repositoryId,
-                                                    @Nullable ZonedDateTime 
startTime, @Nullable ZonedDateTime endTime )
+                                                    ZonedDateTime startTime, 
ZonedDateTime endTime )
         throws MetadataRepositoryException;
 
     /**
@@ -306,7 +302,7 @@ public interface MetadataRepository
      * @since 3.0
      */
     List<ArtifactMetadata> getArtifactsByDateRange( RepositorySession session, 
String repositoryId,
-                                                    @Nullable ZonedDateTime 
startTime, @Nullable ZonedDateTime endTime,
+                                                    ZonedDateTime startTime, 
ZonedDateTime endTime,
                                                     QueryParameter 
queryParameter )
         throws MetadataRepositoryException;
 
@@ -325,7 +321,7 @@ public interface MetadataRepository
      * @since 3.0
      */
     Stream<ArtifactMetadata> getArtifactByDateRangeStream( RepositorySession 
session, String repositoryId,
-                                                           @Nullable 
ZonedDateTime startTime, @Nullable ZonedDateTime endTime )
+                                                           ZonedDateTime 
startTime, ZonedDateTime endTime )
         throws MetadataRepositoryException;
 
     /**
@@ -343,7 +339,7 @@ public interface MetadataRepository
      * @since 3.0
      */
     Stream<ArtifactMetadata> getArtifactByDateRangeStream( RepositorySession 
session, String repositoryId,
-                                                           @Nullable 
ZonedDateTime startTime, @Nullable ZonedDateTime endTime,
+                                                           ZonedDateTime 
startTime, ZonedDateTime endTime,
                                                            QueryParameter 
queryParameter )
         throws MetadataRepositoryException;
 
@@ -371,7 +367,7 @@ public interface MetadataRepository
      * @throws MetadataRepositoryException if the artifact retrieval fails.
      */
     List<ArtifactMetadata> getArtifactsByProjectVersionFacet( 
RepositorySession session, String key, String value,
-                                                              @Nullable String 
repositoryId )
+                                                              String 
repositoryId )
         throws MetadataRepositoryException;
 
     /**
@@ -386,7 +382,7 @@ public interface MetadataRepository
      * @return a list of artifact objects for each artifact that matches the 
search string
      * @throws MetadataRepositoryException if the artifact retrieval fails.
      */
-    List<ArtifactMetadata> getArtifactsByAttribute( RepositorySession session, 
String key, String value, @Nullable String repositoryId )
+    List<ArtifactMetadata> getArtifactsByAttribute( RepositorySession session, 
String key, String value, String repositoryId )
         throws MetadataRepositoryException;
 
     /**
@@ -400,7 +396,7 @@ public interface MetadataRepository
      * @return a list of artifacts or a empty list, if no artifact was found
      * @throws MetadataRepositoryException if the artifact retrieval fails
      */
-    List<ArtifactMetadata> getArtifactsByProjectVersionAttribute( 
RepositorySession session, String key, String value, @Nullable String 
repositoryId )
+    List<ArtifactMetadata> getArtifactsByProjectVersionAttribute( 
RepositorySession session, String key, String value, String repositoryId )
         throws MetadataRepositoryException;
 
     /**
diff --git 
a/archiva-modules/plugins/metadata-store-cassandra/src/main/java/org/apache/archiva/metadata/repository/cassandra/CassandraMetadataRepository.java
 
b/archiva-modules/plugins/metadata-store-cassandra/src/main/java/org/apache/archiva/metadata/repository/cassandra/CassandraMetadataRepository.java
index 8fbdd0e..d350c67 100644
--- 
a/archiva-modules/plugins/metadata-store-cassandra/src/main/java/org/apache/archiva/metadata/repository/cassandra/CassandraMetadataRepository.java
+++ 
b/archiva-modules/plugins/metadata-store-cassandra/src/main/java/org/apache/archiva/metadata/repository/cassandra/CassandraMetadataRepository.java
@@ -71,7 +71,6 @@ import org.modelmapper.ModelMapper;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import javax.annotation.ParametersAreNonnullByDefault;
 import java.time.Instant;
 import java.time.ZonedDateTime;
 import java.util.*;
@@ -89,7 +88,6 @@ import static 
org.apache.archiva.metadata.repository.cassandra.model.ColumnNames
  * @author Olivier Lamy
  * @since 2.0.0
  */
-@ParametersAreNonnullByDefault
 public class CassandraMetadataRepository
     extends AbstractMetadataRepository implements MetadataRepository
 {
diff --git 
a/archiva-modules/plugins/metadata-store-file/src/main/java/org/apache/archiva/metadata/repository/file/FileMetadataRepository.java
 
b/archiva-modules/plugins/metadata-store-file/src/main/java/org/apache/archiva/metadata/repository/file/FileMetadataRepository.java
index a344064..00e68c1 100644
--- 
a/archiva-modules/plugins/metadata-store-file/src/main/java/org/apache/archiva/metadata/repository/file/FileMetadataRepository.java
+++ 
b/archiva-modules/plugins/metadata-store-file/src/main/java/org/apache/archiva/metadata/repository/file/FileMetadataRepository.java
@@ -46,7 +46,6 @@ import org.apache.commons.lang3.StringUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import javax.annotation.ParametersAreNonnullByDefault;
 import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.io.InputStream;
@@ -82,7 +81,6 @@ import java.util.stream.Stream;
  *
  * Some retrieval methods may not be very efficient.
  */
-@ParametersAreNonnullByDefault
 public class FileMetadataRepository
         extends AbstractMetadataRepository implements MetadataRepository {
 
diff --git 
a/archiva-modules/plugins/metadata-store-jcr/src/main/java/org/apache/archiva/metadata/repository/jcr/JcrMetadataRepository.java
 
b/archiva-modules/plugins/metadata-store-jcr/src/main/java/org/apache/archiva/metadata/repository/jcr/JcrMetadataRepository.java
index d803d48..2e09bbf 100644
--- 
a/archiva-modules/plugins/metadata-store-jcr/src/main/java/org/apache/archiva/metadata/repository/jcr/JcrMetadataRepository.java
+++ 
b/archiva-modules/plugins/metadata-store-jcr/src/main/java/org/apache/archiva/metadata/repository/jcr/JcrMetadataRepository.java
@@ -35,7 +35,6 @@ import org.apache.jackrabbit.commons.cnd.ParseException;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import javax.annotation.ParametersAreNonnullByDefault;
 import javax.jcr.*;
 import javax.jcr.query.*;
 import java.io.IOException;
@@ -57,7 +56,6 @@ import static 
org.apache.archiva.metadata.repository.jcr.JcrConstants.*;
  * TODO below: revise storage format for project version metadata
  * TODO revise reference storage
  */
-@ParametersAreNonnullByDefault
 public class JcrMetadataRepository
         extends AbstractMetadataRepository implements MetadataRepository, 
RepositoryStatisticsProvider {
 

Reply via email to