[GitHub] madlib pull request #288: Jira:1239: Converts features from multiple columns...

[iyerr3]

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

    https://github.com/apache/madlib/pull/288#discussion_r199981733
  
    --- Diff: src/config/Modules.yml ---
    @@ -50,3 +50,4 @@ modules:
         - name: validation
           depends: ['array_ops', 'regress']
         - name: stemmer
    +    - name: cols_vec
    --- End diff --
    
    I'm not convinced that we need a new module for this functionality. IMO 
this is better suited for the `utilities` module as a separate file.  


---