GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/9726
[SPARK-11750][SQL] revert SPARK-11727 and code clean up
After some experiment, I found it's not convenient to have separate encoder
builders: `FlatEncoder` and `ProductEncoder`. For example, when create encoders
for `ScalaUDF`, we have no idea if the type `T` is flat or not. So I revert the
splitting change in https://github.com/apache/spark/pull/9693, while still
keeping the bug fixes and tests.
This PR also did some code clean up, like removing `trait ScalaReflection`
and put all methods just in `object ScalaReflection`.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/cloud-fan/spark follow
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/9726.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 #9726
----
commit 6dbc1247f6f4a3a0ac50828ec64672a793a76c7b
Author: Wenchen Fan <[email protected]>
Date: 2015-11-16T05:04:26Z
revert #11727
----
---
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]