GitHub user davies opened a pull request:

    https://github.com/apache/spark/pull/7003

    [SPARK-8610] [SQL] Separate Row and InternalRow (part 2)

    Currently, we use GenericRow both for Row and InternalRow, which is 
confusing because it could contain Scala type also Catalyst types.
    
    This PR changes to use GenericInternalRow for InternalRow (contains 
catalyst types), GenericRow for Row (contains Scala types).
    
    Also fixes some incorrect use of InternalRow or Row.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/davies/spark internalrow

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/7003.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 #7003
    
----
commit 9d243509c3f38dd42e079318eee1a653cdf5cfec
Author: Davies Liu <[email protected]>
Date:   2015-06-24T23:37:53Z

    separate Row and InternalRow (part 2)

commit 288b31fe0b931854e5a984506cd6756f157fc01a
Author: Davies Liu <[email protected]>
Date:   2015-06-24T23:38:52Z

    Merge branch 'master' of github.com:apache/spark into internalrow
    
    Conflicts:
        
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/generators.scala

----


---
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