Github user viirya commented on the pull request:

    https://github.com/apache/spark/pull/4994#issuecomment-81108144
  
    Because `updateProjection` is the projection between `child.output` and the 
aggregations `updateExpressions`. Its `inputSchema` should be just 
`child.output`. It is unnecessary to add `computeFunctions.flatMap(_.schema)`.
     
    It would not break anything because it just adds useless input schema. The 
aggregations `updateExpressions` will never use that. That is why removing it 
passes all tests.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to