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

    https://github.com/apache/madlib/pull/291#discussion_r205920988
  
    --- 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 --
    
    Well you don't need this function - you might need `::TEXT`. But I believe 
we decided that we'll let the platform fail if the array cannot be built by it. 
Wouldn't this build a successful array when the platform is failing. 


---

Reply via email to