[geode] branch feature/GEODE-3781 updated: added comment

[ds...@apache.org]

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

dschneider pushed a commit to branch feature/GEODE-3781
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/feature/GEODE-3781 by this push:
     new f20c3a6  added comment
f20c3a6 is described below

commit f20c3a6e82dd294323a68f1acd06e84f369552fc
Author: Darrel Schneider <ds...@pivotal.io>
AuthorDate: Wed Nov 1 12:27:45 2017 -0700

    added comment
---
 .../src/main/java/org/apache/geode/connectors/jdbc/JDBCLoader.java  | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/JDBCLoader.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/JDBCLoader.java
index 5219576..162d053 100644
--- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/JDBCLoader.java
+++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/JDBCLoader.java
@@ -36,7 +36,13 @@ public class JDBCLoader<K, V> implements CacheLoader<K, V> {
 
   @SuppressWarnings("unchecked")
   @Override
+  /**
+   * @return this method always returns a PdxInstance. It does not matter what the V generic
+   *         parameter is set to.
+   */
   public V load(LoaderHelper<K, V> helper) throws CacheLoaderException {
+    // The following cast to V is to keep the compiler happy
+    // but is erased at runtime and no actual cast happens.
     return (V) this.manager.read(helper.getRegion(), helper.getKey());
   }
 

-- 
To stop receiving notification emails like this one, please contact
['"commits@geode.apache.org" <co...@geode.apache.org>'].