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

    https://github.com/apache/madlib/pull/291#discussion_r205897628
  
    --- 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 --
    
    I'm not clear on the need for `is_forced`. Are there platforms that have a 
bool-to-text cast require this patching? 


---

Reply via email to