Github user WeichenXu123 commented on the issue:

    https://github.com/apache/spark/pull/17819
  
    Yes you can only move `setInputCols` into the outer class to resolve this 
issue. But I prefer merge it together. I think we can unify the `transform` 
method. (First we check param `inputCol` and `inputCols`, and construct a col 
list and pass it to the transform code. it will simplify the code).


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to