Github user alberskib commented on the pull request:
https://github.com/apache/spark/pull/9642#issuecomment-156694329
Hey @hvanhovell ! Thanks for the comment. I agree with you that if we want
to introduce this change we need to take care of `UnsafeRow` and `SpecificRow`
as well.
I have couple of questions for you:
> I'd actualy rather not touch this at all ...
You mean discard change and only update documentation? Yes this is one
option - I even think that it is not so bad.
On the other hand it could be misleading and many users will not notice the
difference in documentation - this leads them to making the same errors. I
think that right now there is lot of code that is calculating wrong outcomes as
a result of this bug.
> I can currently think of only one place in which this causes some problems
You mean problem caused by not introducing this change?
I think that we should analyse pros and cons and decide how to proceed.
---
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]
