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

    https://github.com/apache/madlib/pull/291#discussion_r205914522
  
    --- Diff: src/ports/postgres/modules/utilities/validate_args.py_in ---
    @@ -513,11 +513,12 @@ def array_col_has_same_dimension(tbl, col):
     # ------------------------------------------------------------------------
     
     
    -def explicit_bool_to_text(tbl, cols, schema_madlib):
    +def explicit_bool_to_text(tbl, cols, schema_madlib, is_forced=False):
    --- End diff --
    
    w/ @ArvindSridhar On platforms that has bool to text casting (gpdb5, pg 
9.6, pg 10), we still need this to make sure we can create an array of bool and 
text types.


---

Reply via email to