GitHub user jingyimei opened a pull request:

    https://github.com/apache/madlib/pull/273

    Minibatch Preprocessing: fix dependent var with special character

    JIRA:MADLIB-1237
    
    Previously, minibatch processing would error out when the specified
    dependent variable has special characters within its values. We
    fixed this in two places:
    1. in the query with WHERE condition, we use $$ to queto string instead
       of ' ' to do string equals.
    2. in the query with creating an array column, instead of using
       SELECT '{ele'with*special_char, 'M,M', 'M$M'}'::text[], we call
       SELECT string_to_array(''ele'with*special_char', 'M"M', 'M$M'', 
',')::text[]
    
    Install check test cases also get updated.
    Co-Authored-by: Jingyi Mei <[email protected]>

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/madlib/madlib bug_minibatch_preprocessor

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/madlib/pull/273.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #273
    
----
commit 3a55cafd7fad1e87996b4584f7e0431529da7a52
Author: Jingyi Mei <jmei@...>
Date:   2018-05-23T23:29:54Z

    Minibatch Preprocessing: fix dependent var with special character
    
    JIRA:MADLIB-1237
    
    Previously, minibatch processing would error out when the specified
    dependent variable has special characters within its values. We
    fixed this in two places:
    1. in the query with WHERE condition, we use $$ to queto string instead
       of ' ' to do string equals.
    2. in the query with creating an array column, instead of using
       SELECT '{ele'with*special_char, 'M,M', 'M$M'}'::text[], we call
       SELECT string_to_array(''ele'with*special_char', 'M"M', 'M$M'', 
',')::text[]
    
    Install check test cases also get updated.
    Co-Authored-by: Jingyi Mei <[email protected]>

----


---

Reply via email to