Github user kaspersorensen commented on a diff in the pull request:

    https://github.com/apache/metamodel/pull/86#discussion_r49181994
  
    --- Diff: 
mongodb/common/src/main/java/org/apache/metamodel/mongodb/common/MongoDBUtils.java
 ---
    @@ -73,5 +101,29 @@ private static Object toValue(Column column, Object 
value) {
             }
             return value;
         }
    +    
    +    public static Object covertArrayToList(Object arr) {
    --- End diff --
    
    Could we make this method private and just call it in the toValue(...) 
method instead? Since the method is really restricted in it's use it seems 
maybe we can add an ```if``` above and then not worry about other people 
misunderstanding what this is used for.
    
    Small note on naming - it's misspelled (missing "n" in convert).


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to