GitHub user cloud-fan opened a pull request: https://github.com/apache/spark/pull/8613
[SPARK-10461][SQL] make sure `input.primitive` is always variable name not code at `GenerateUnsafeProjection` When we generate unsafe code inside `createCodeForXXX`, we always assign the `input.primitive` to a temp variable in case `input.primitive` is expression code. This PR did some refactor to make sure `input.primitive` is always variable name, and some other typo and style fixes. You can merge this pull request into a Git repository by running: $ git pull https://github.com/cloud-fan/spark minor Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/8613.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #8613 ---- commit 39b6d029caf35476020216160e137327ae7a21df Author: Wenchen Fan <cloud0...@outlook.com> Date: 2015-08-28T05:44:52Z make sure input.primitive is always variable name not code ---- --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org