[geode] branch feature/GEODE-6291 updated: WIP: added some comments about how to do this story

[ds...@apache.org]

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 6c1293b  WIP: added some comments about how to do this story
6c1293b is described below

commit 6c1293bb7b7cb4cb54e846ac9c2d28401fc9f40f
Author: Darrel Schneider <ds...@pivotal.io>
AuthorDate: Thu Jan 17 18:12:31 2019 -0800

    WIP: added some comments about how to do this story
---
 .../connectors/jdbc/internal/cli/CreateMappingCommand.java   | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateMappingCommand.java b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateMappingCommand.java
index 1412f41..784d492 100644
--- a/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateMappingCommand.java
+++ b/geode-connectors/src/main/java/org/apache/geode/connectors/jdbc/internal/cli/CreateMappingCommand.java
@@ -125,6 +125,18 @@ public class CreateMappingCommand extends SingleGfshCommand {
       return ResultModel.createError(ex.getMessage());
     }
 
+    // Do the following on one and only one targetMember:
+    // 1. connect to the database (using dataSourceName)
+    // 2. read the table meta data
+    // 3. generates a pdx type using PdxInstanceFactory with null field values
+    // 4. return the field mapping info
+    // The field mapping info will be passed to CreateMappingFunction
+    // The field mapping will have the following:
+    // 1. pdx field name
+    // 2. pdx field type
+    // 3. sql column name
+    // 4. sql column type
+
     // action
     Object[] arguments = new Object[] {mapping, synchronous};
     List<CliFunctionResult> results =