Github user MaxGekk commented on the issue:

    https://github.com/apache/spark/pull/21699
  
    >  Were you planning to add a new overload for each existing String 
version, e.g. pivot(Column) and pivot(Column, java.util.List[Any])?
    
    The methods have been added already. @rednaxelafx Please, look at the lines:
    
https://github.com/apache/spark/pull/21699/files#diff-95bb2228c67e3cce4c729e44e2d82422R362
    
https://github.com/apache/spark/pull/21699/files#diff-95bb2228c67e3cce4c729e44e2d82422R377
    
    > Yes you've already included that in test case examples so that's already 
good
    
    I added this test case: 
https://github.com/apache/spark/pull/21699/files#diff-50aa7d3b7b7934a7df6f414396e74c3cR271
 . Also I am trying to find a case when existing API with `String` doesn't work 
well (some `Column` expressions maybe). 


---

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

Reply via email to