Merge pull request #61 from graphql-java/update-pr-about-dispatch-counts

The PR 56 followup work

Author: bbakerman

build.gradle

@@ -61,5 +61,5 @@ dependencies {
 
 task wrapper(type: Wrapper) {
     gradleVersion = '4.0'
-    distributionUrl = "http://services.gradle.org/distributions/gradle-4.0-all.zip"
+    distributionUrl = "https://services.gradle.org/distributions/gradle-4.0-all.zip"
 }

gradle/publishing.gradle

@@ -4,7 +4,7 @@ apply plugin: 'com.jfrog.bintray'
 publishing {
     repositories {
         maven {
-            url 'http://dl.bintray.com/bbakerman/java-dataloader'
+            url 'https://dl.bintray.com/bbakerman/java-dataloader'
         }
     }
 }
@@ -52,7 +52,7 @@ publishing {
                     licenses {
                         license {
                             name 'The Apache Software License, Version 2.0'
-                            url 'http://www.apache.org/licenses/LICENSE-2.0.txt'
+                            url 'https://www.apache.org/licenses/LICENSE-2.0.txt'
                             distribution 'repo'
                         }
                     }

gradle/wrapper/gradle-wrapper.properties

@@ -3,4 +3,4 @@ distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists
-distributionUrl=http\://services.gradle.org/distributions/gradle-4.0-all.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-4.0-all.zip

src/main/java/org/dataloader/BatchLoader.java

@@ -71,6 +71,7 @@
  * @author <a href="https://github.com/bbakerman/">Brad Baker</a>
  */
 @FunctionalInterface
+@PublicSpi
 public interface BatchLoader<K, V> {
 
     /**

src/main/java/org/dataloader/BatchLoaderContextProvider.java

@@ -5,6 +5,7 @@
  * provide overall calling context to the {@link org.dataloader.BatchLoader} call.  A common use
  * case is for propagating user security credentials or database connection parameters for example.
  */
+@PublicSpi
 public interface BatchLoaderContextProvider {
     /**
      * @return a context object that may be needed in batch load calls

src/main/java/org/dataloader/BatchLoaderEnvironment.java

@@ -12,6 +12,7 @@
  * This object is passed to a batch loader as calling context.  It could contain security credentials
  * of the calling users for example or database parameters that allow the data layer call to succeed.
  */
+@PublicApi
 public class BatchLoaderEnvironment {
 
     private final Object context;

src/main/java/org/dataloader/BatchLoaderEnvironmentProvider.java

@@ -6,6 +6,7 @@
  * the {@link org.dataloader.BatchLoader} call.  A common use
  * case is for propagating user security credentials or database connection parameters.
  */
+@PublicSpi
 public interface BatchLoaderEnvironmentProvider {
     /**
      * @return a {@link org.dataloader.BatchLoaderEnvironment} that may be needed in batch calls

src/main/java/org/dataloader/BatchLoaderWithContext.java

@@ -11,6 +11,7 @@
  * See {@link org.dataloader.BatchLoader} for more details on the design invariants that you must implement in order to
  * use this interface.
  */
+@PublicSpi
 public interface BatchLoaderWithContext<K, V> {
     /**
      * Called to batch load the provided keys and return a promise to a list of values.  This default

src/main/java/org/dataloader/CacheMap.java

@@ -35,6 +35,7 @@
  * @author <a href="https://github.com/aschrijver/">Arnold Schrijver</a>
  * @author <a href="https://github.com/bbakerman/">Brad Baker</a>
  */
+@PublicSpi
 public interface CacheMap<U, V> {
 
     /**

src/main/java/org/dataloader/DataLoader.java

@@ -54,6 +54,7 @@
  * @author <a href="https://github.com/aschrijver/">Arnold Schrijver</a>
  * @author <a href="https://github.com/bbakerman/">Brad Baker</a>
  */
+@PublicApi
 public class DataLoader<K, V> {
 
     private final DataLoaderHelper<K, V> helper;
@@ -470,7 +471,7 @@ public CompletableFuture<List<V>> loadMany(List<K> keys, List<Object> keyContext
      * @return the promise of the queued load requests
      */
     public CompletableFuture<List<V>> dispatch() {
-        return helper.dispatch().futureList;
+        return helper.dispatch().getPromisedResults();
     }
 
     /**
@@ -480,9 +481,9 @@ public CompletableFuture<List<V>> dispatch() {
      * If batching is disabled, or there are no queued requests, then a succeeded promise with no entries dispatched is
      * returned.
      *
-     * @return the promise of the queued load requests and the number of entries dispatched.
+     * @return the promise of the queued load requests and the number of keys dispatched.
      */
-    public DataLoaderHelper.DispatchResult<V> dispatchWithCounts() {
+    public DispatchResult<V> dispatchWithCounts() {
         return helper.dispatch();
     }