[DE-462] removed hash and skipList index

Author: rashtao

core/src/main/java/com/arangodb/ArangoCollection.java

@@ -610,34 +610,6 @@ <T> MultiDocumentEntity<DocumentDeleteEntity<T>> deleteDocuments(
      */
     String deleteIndex(String id);
 
-    /**
-     * Creates a hash index for the collection if it does not already exist.
-     *
-     * @param fields  A list of attribute paths
-     * @param options Additional options, can be null
-     * @return information about the index
-     * @see
-     * <a href="https://www.arangodb.com/docs/stable/http/indexes-hash.html#create-hash-index">API Documentation</a>
-     * @deprecated use {@link #ensurePersistentIndex(Iterable, PersistentIndexOptions)} instead. Since ArangoDB 3.7 a
-     * hash index is an alias for a persistent index.
-     */
-    @Deprecated
-    IndexEntity ensureHashIndex(Iterable<String> fields, HashIndexOptions options);
-
-    /**
-     * Creates a skip-list index for the collection, if it does not already exist.
-     *
-     * @param fields  A list of attribute paths
-     * @param options Additional options, can be null
-     * @return information about the index
-     * @see <a href="https://www.arangodb.com/docs/stable/http/indexes-skiplist.html#create-skip-list">API
-     * Documentation</a>
-     * @deprecated use {@link #ensurePersistentIndex(Iterable, PersistentIndexOptions)} instead. Since ArangoDB 3.7 a
-     * skiplist index is an alias for a persistent index.
-     */
-    @Deprecated
-    IndexEntity ensureSkiplistIndex(Iterable<String> fields, SkiplistIndexOptions options);
-
     /**
      * Creates a persistent index for the collection, if it does not already exist.
      *

core/src/main/java/com/arangodb/async/ArangoCollectionAsync.java

@@ -650,36 +650,6 @@ <T> CompletableFuture<MultiDocumentEntity<DocumentDeleteEntity<T>>> deleteDocume
      */
     CompletableFuture<String> deleteIndex(final String id);
 
-    /**
-     * Creates a hash index for the collection, if it does not already exist.
-     *
-     * @param fields  A list of attribute paths
-     * @param options Additional options, can be null
-     * @return information about the index
-     * @see
-     * <a href="https://www.arangodb.com/docs/stable/http/indexes-hash.html#create-hash-index">API Documentation</a>
-     * @deprecated use {@link #ensurePersistentIndex(Iterable, PersistentIndexOptions)} instead. Since ArangoDB 3.7 a
-     * hash index is an alias for a persistent index.
-     */
-    @Deprecated
-    CompletableFuture<IndexEntity> ensureHashIndex(final Iterable<String> fields, final HashIndexOptions options);
-
-    /**
-     * Creates a skip-list index for the collection, if it does not already exist.
-     *
-     * @param fields  A list of attribute paths
-     * @param options Additional options, can be null
-     * @return information about the index
-     * @see <a href="https://www.arangodb.com/docs/stable/http/indexes-skiplist.html#create-skip-list">API
-     * Documentation</a>
-     * @deprecated use {@link #ensurePersistentIndex(Iterable, PersistentIndexOptions)} instead. Since ArangoDB 3.7 a
-     * skiplist index is an alias for a persistent index.
-     */
-    @Deprecated
-    CompletableFuture<IndexEntity> ensureSkiplistIndex(
-            final Iterable<String> fields,
-            final SkiplistIndexOptions options);
-
     /**
      * Creates a persistent index for the collection, if it does not already exist.
      *

core/src/main/java/com/arangodb/async/internal/ArangoCollectionAsyncImpl.java

@@ -361,22 +361,6 @@ public CompletableFuture<String> deleteIndex(final String id) {
         return executor.execute(deleteIndexRequest(id), deleteIndexResponseDeserializer());
     }
 
-    @Override
-    @Deprecated
-    public CompletableFuture<IndexEntity> ensureHashIndex(
-            final Iterable<String> fields,
-            final HashIndexOptions options) {
-        return executor.execute(createHashIndexRequest(fields, options), IndexEntity.class);
-    }
-
-    @Override
-    @Deprecated
-    public CompletableFuture<IndexEntity> ensureSkiplistIndex(
-            final Iterable<String> fields,
-            final SkiplistIndexOptions options) {
-        return executor.execute(createSkiplistIndexRequest(fields, options), IndexEntity.class);
-    }
-
     @Override
     public CompletableFuture<IndexEntity> ensurePersistentIndex(
             final Iterable<String> fields,

core/src/main/java/com/arangodb/internal/ArangoCollectionImpl.java

@@ -359,18 +359,6 @@ public String deleteIndex(final String id) {
         return executor.execute(deleteIndexRequest(id), deleteIndexResponseDeserializer());
     }
 
-    @Deprecated
-    @Override
-    public IndexEntity ensureHashIndex(final Iterable<String> fields, final HashIndexOptions options) {
-        return executor.execute(createHashIndexRequest(fields, options), IndexEntity.class);
-    }
-
-    @Deprecated
-    @Override
-    public IndexEntity ensureSkiplistIndex(final Iterable<String> fields, final SkiplistIndexOptions options) {
-        return executor.execute(createSkiplistIndexRequest(fields, options), IndexEntity.class);
-    }
-
     @Override
     public IndexEntity ensurePersistentIndex(final Iterable<String> fields, final PersistentIndexOptions options) {
         return executor.execute(createPersistentIndexRequest(fields, options), IndexEntity.class);

core/src/main/java/com/arangodb/internal/InternalArangoCollection.java

@@ -425,25 +425,6 @@ private String createIndexId(final String id) {
         return index;
     }
 
-    @Deprecated
-    protected InternalRequest createHashIndexRequest(final Iterable<String> fields, final HashIndexOptions options) {
-        final InternalRequest request = request(db.dbName(), RequestType.POST, PATH_API_INDEX);
-        request.putQueryParam(COLLECTION, name);
-        request.setBody(
-                getSerde().serialize(OptionsBuilder.build(options != null ? options : new HashIndexOptions(), fields)));
-        return request;
-    }
-
-    @Deprecated
-    protected InternalRequest createSkiplistIndexRequest(final Iterable<String> fields, final SkiplistIndexOptions options) {
-        final InternalRequest request = request(db.dbName(), RequestType.POST, PATH_API_INDEX);
-        request.putQueryParam(COLLECTION, name);
-        request.setBody(
-                getSerde().serialize(OptionsBuilder.build(options != null ? options : new SkiplistIndexOptions(),
-                        fields)));
-        return request;
-    }
-
     protected InternalRequest createPersistentIndexRequest(
             final Iterable<String> fields, final PersistentIndexOptions options) {
         final InternalRequest request = request(db.dbName(), RequestType.POST, PATH_API_INDEX);

core/src/main/java/com/arangodb/model/HashIndexOptions.java

@@ -41,24 +41,6 @@ public static UserCreateOptions build(final UserCreateOptions options, final Str
         return options.user(user).passwd(passwd);
     }
 
-    /**
-     * @deprecated use {@link #build(PersistentIndexOptions, Iterable)} instead. Since ArangoDB 3.7 a hash index is an
-     * alias for a persistent index.
-     */
-    @Deprecated
-    public static HashIndexOptions build(final HashIndexOptions options, final Iterable<String> fields) {
-        return options.fields(fields);
-    }
-
-    /**
-     * @deprecated use {@link #build(PersistentIndexOptions, Iterable)} instead. Since ArangoDB 3.7 a skiplist index is
-     * an alias for a persistent index.
-     */
-    @Deprecated
-    public static SkiplistIndexOptions build(final SkiplistIndexOptions options, final Iterable<String> fields) {
-        return options.fields(fields);
-    }
-
     public static PersistentIndexOptions build(final PersistentIndexOptions options, final Iterable<String> fields) {
         return options.fields(fields);
     }