Improve the behaviour of the APIs (now somewhat redundant) holder refresh method

api/src/main/java/me/lucko/luckperms/api/PermissionHolder.java

@@ -84,6 +84,9 @@ public interface PermissionHolder {
     /**
      * Refreshes and applies any changes to the cached holder data.
      *
+     * <p>Calling this method is unnecessary in most cases. Cache updates are handled
+     * behind the scenes by the implementation.</p>
+     *
      * @return the task future
      * @since 4.0
      */

common/src/main/java/me/lucko/luckperms/common/api/delegates/model/ApiPermissionHolder.java

@@ -99,7 +99,7 @@ public class ApiPermissionHolder implements me.lucko.luckperms.api.PermissionHol
     @Nonnull
     @Override
     public CompletableFuture<Void> refreshCachedData() {
-        return this.handle.getCachedData().reloadAll();
+        return CompletableFuture.runAsync(() -> this.handle.getCachedData().invalidateCaches());
     }
 
     @Nonnull