madlib-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From fmcquillan99 <...@git.apache.org>
Subject [GitHub] madlib issue #234: Create lower case column name in encode_categorical_varia...
Date Fri, 16 Feb 2018 22:55:03 GMT
Github user fmcquillan99 commented on the issue:

    https://github.com/apache/madlib/pull/234
  
    ```
    DROP TABLE IF EXISTS abalone_out, abalone_out_dictionary;
    SELECT madlib.encode_categorical_variables (
            'abalone',                   -- Source table
            'abalone_out',               -- Output table
            'height>.10'               -- Categorical columns
            );
    ```
    is failing now:
    ```
    Done.
    (psycopg2.ProgrammingError) plpy.SPIError: operator does not exist: double precision >
text
    LINE 1: SELECT array_agg(DISTINCT height>.10::TEXT) AS "height>.10" ...
                                            ^
    HINT:  No operator matches the given name and argument type(s). You might need to add
explicit type casts.
    QUERY:  SELECT array_agg(DISTINCT height>.10::TEXT) AS "height>.10"  FROM abalone
    CONTEXT:  Traceback (most recent call last):
      PL/Python function "encode_categorical_variables", line 23, in <module>
        return encode_categorical.encode_categorical_variables(**globals())
      PL/Python function "encode_categorical_variables", line 608, in encode_categorical_variables
      PL/Python function "encode_categorical_variables", line 103, in build_output_table
      PL/Python function "encode_categorical_variables", line 542, in _get_distinct_values
    PL/Python function "encode_categorical_variables"
     [SQL: "SELECT madlib.encode_categorical_variables (\n        'abalone',             
     -- Source table\n        'abalone_out',               -- Output table\n        'height>.10'
              -- Categorical columns\n        );"]
    ```
    whereas this used to work before in 1.13, i.e., it used to produce 2 columns called
    ```
    height>.10_False 
    ```
    and
    ```
    height>.10_True
    ```
    In this PR I expected to see
    ```
    height>.10_false 
    ```
    and
    ```
    height>.10_true
    ```



---

Mime
View raw message