[SPARK-9361] Refactor new aggregation code to reduce the times of checking compatibility

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.5.0
• Component/s: SQL
• Assignee: L. C. Hsieh
• Reporter: L. C. Hsieh
• Created: 26/Jul/15 16:53
• Updated: 30/Jul/15 17:32
• Resolved: 30/Jul/15 17:32

Description

Currently, we call aggregate.Utils.tryConvert in many places to check it the logical.aggregate can be run with new aggregation. But looks like aggregate.Utils.tryConvert costs much time to run. We should only call tryConvert once and keep it value in logical.aggregate and reuse it.

Issue Links

• links to: [Github] Pull Request #7677 (viirya)