Github user pgandhi999 commented on the issue:

    https://github.com/apache/spark/pull/22144
  
    @cloud-fan I see a flag supportsPartial which was removed in PR 
https://github.com/apache/spark/pull/16461. Is this the flag you were talking 
about? If so, I was thinking that we make this flag configurable from the user 
end, so it does not change any existing stuff but will create a backward 
compatibility for users whose custom UDAF's might not support partial 
aggregation like the one in the example mentioned in the PR. Am I thinking in 
the right direction? Would really appreciate your guidance in this matter. 
Thank you.


---

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

Reply via email to