Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11002
---
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 ena
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/11002#issuecomment-179685420
LGTM, merging this into master.
---
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
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/11002#issuecomment-179584171
@davies Can you take a look if this is proper to merge? Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/11002#issuecomment-178980874
cc @davies @JoshRosen
---
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
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/11002#issuecomment-178968321
Hmm, I just found this after another check.
---
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 proje
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/11002#issuecomment-178369142
I will check again. But yesterday I just found this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If y
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11002#issuecomment-178337808
Seems OK, are there other redundant usages? The constant name including 13
seems overly specific too
---
If your project is set up for it, you can reply to this email a
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/11002#issuecomment-178160294
This change looks good to me, maybe @JoshRosen who was the last to touch
that line can take a look?
---
If your project is set up for it, you can reply to this email a