[geode] branch feature/GEODE-6291 updated: getColumnNameForField now remembers its answer to speed up future calls

[dschneider]

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

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


The following commit(s) were added to refs/heads/feature/GEODE-6291 by this 
push:
     new e20fdb1  getColumnNameForField now remembers its answer to speed up 
future calls
e20fdb1 is described below

commit e20fdb186f5ff1f913ca83a800049fd19d407d90
Author: Darrel Schneider <dschnei...@pivotal.io>
AuthorDate: Tue Jan 29 13:02:56 2019 -0800

    getColumnNameForField now remembers its answer to speed up future calls
---
 .../geode/connectors/jdbc/internal/configuration/RegionMapping.java     | 2 ++
 1 file changed, 2 insertions(+)

diff --git 
a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/configuration/RegionMapping.java
 
b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/configuration/RegionMapping.java
index 6f28904..48a446d 100644
--- 
a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/configuration/RegionMapping.java
+++ 
b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/configuration/RegionMapping.java
@@ -195,6 +195,7 @@ public class RegionMapping implements CacheElement {
     }
     exactMatch = getFieldMappingByJdbcName(fieldName);
     if (exactMatch != null) {
+      this.pdxToFieldMappings.put(fieldName, exactMatch);
       return exactMatch.getJdbcName();
     }
     FieldMapping inexactMatch = null;
@@ -210,6 +211,7 @@ public class RegionMapping implements CacheElement {
     if (inexactMatch == null) {
       throw new JdbcConnectorException("No column matched the pdx field \"" + 
fieldName + "\".");
     }
+    this.pdxToFieldMappings.put(fieldName, inexactMatch);
     return inexactMatch.getJdbcName();
   }