feat(deleteByQuery): Implement deleteBy, deprecate old method

Author: PLNech

algoliasearch/src/main/java/com/algolia/search/saas/Index.java

@@ -479,7 +479,6 @@ public Request getObjectsAsync(final @NonNull List<String> objectIDs, final List
      * @param taskID            Identifier of the task (as returned by the server).
      * @param completionHandler The listener that will be notified of the request's outcome.
      * @return A cancellable request.
-     *
      * @deprecated Task IDs are always integers. Please use {@link #waitTaskAsync(int, CompletionHandler)} instead.
      */
     public Request waitTaskAsync(final @NonNull String taskID, @NonNull CompletionHandler completionHandler) {
@@ -541,11 +540,12 @@ public Request deleteObjectsAsync(final @NonNull List<String> objectIDs, Complet
     }
 
     /**
-     * Delete all objects matching a query (helper).
+     * Delete all objects matching a query (helper) using browse and deleteObjects.
      *
      * @param query             The query that objects to delete must match.
      * @param completionHandler The listener that will be notified of the request's outcome.
      * @return A cancellable request.
+     * @deprecated use {@link Index#deleteByAsync(Query, CompletionHandler)} instead.
      */
     public Request deleteByQueryAsync(@NonNull Query query, CompletionHandler completionHandler) {
         final Query queryCopy = new Query(query);
@@ -558,6 +558,23 @@ public Request deleteByQueryAsync(@NonNull Query query, CompletionHandler comple
         }.start();
     }
 
+    /**
+     * Delete all objects matching a query (helper).
+     *
+     * @param query             The query that objects to delete must match.
+     * @param completionHandler The listener that will be notified of the request's outcome.
+     * @return A cancellable request.
+     */
+    public Request deleteByAsync(@NonNull Query query, CompletionHandler completionHandler) {
+        final Query queryCopy = new Query(query);
+        return getClient().new AsyncTaskRequest(completionHandler) {
+            @NonNull
+            @Override protected JSONObject run() throws AlgoliaException {
+                return deleteBy(queryCopy);
+            }
+        }.start();
+    }
+
     /**
      * Get this index's settings (asynchronously).
      *
@@ -960,11 +977,13 @@ protected JSONObject deleteObjects(List<String> objects) throws AlgoliaException
     }
 
     /**
-     * Delete all objects matching a query
+     * Delete all objects matching a query using browse and deleteObjects.
      *
      * @param query the query string
      * @throws AlgoliaException
+     * @deprecated use {@link Index#deleteBy(Query)} instead.
      */
+    @Deprecated
     protected void deleteByQuery(@NonNull Query query) throws AlgoliaException {
         try {
             boolean hasMore;
@@ -990,6 +1009,21 @@ protected void deleteByQuery(@NonNull Query query) throws AlgoliaException {
         }
     }
 
+    /**
+     * Delete all records matching the query.
+     *
+     * @param query the query string
+     * @throws AlgoliaException
+     */
+    protected JSONObject deleteBy(@NonNull Query query) throws AlgoliaException {
+        try {
+            return client.postRequest("/1/indexes/" + indexName + "/deleteByQuery", new JSONObject().put("params", query.build()).toString(), false);
+        } catch (JSONException e) {
+            e.printStackTrace();
+            return null;
+        }
+    }
+
     /**
      * Search inside the index
      *