[GitHub] [spark] viirya commented on pull request #29404: [SPARK-32018][SQL][FollowUp][3.0] Throw exception on decimal value overflow of sum aggregation

2020-08-12 Thread GitBox
viirya commented on pull request #29404: URL: https://github.com/apache/spark/pull/29404#issuecomment-672633918 And looks like we don't have many choices. I think correctness should be considered first. This is an automated

[GitHub] [spark] viirya commented on pull request #29404: [SPARK-32018][SQL][FollowUp][3.0] Throw exception on decimal value overflow of sum aggregation

2020-08-11 Thread GitBox
viirya commented on pull request #29404: URL: https://github.com/apache/spark/pull/29404#issuecomment-672622923 > However, the aggregation buffer is written in steaming checkpoints. Thus, we can't change to aggregation buffer to resolve the issue. Is this saying the `isEmpty` in