Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11657#issuecomment-196260899
I realize I forgot to also change instances of `getBytes("UTF-8")` to use
the constant, but this isn't any correctness problem, so I might just leave
that for now.
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11657
---
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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11657#issuecomment-196129875
Thanks - merging in 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 have
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/11657#issuecomment-195613637
LGTM. The original idea was to import `c.g.c.b.Charsets.UTF_8` so that
later you could just change the import to
`java.nio.charset.StandardCharsets.UTF_8` without
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11657#issuecomment-195407370
Merged build finished. Test PASSed.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11657#issuecomment-195407372
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11657#issuecomment-195406759
**[Test build #52925 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52925/consoleFull)**
for PR 11657 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11657#issuecomment-195397133
Merged build finished. Test PASSed.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11657#issuecomment-195397138
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11657#issuecomment-195396694
**[Test build #52924 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52924/consoleFull)**
for PR 11657 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11657#issuecomment-195347967
**[Test build #52925 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52925/consoleFull)**
for PR 11657 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11657#issuecomment-195346081
**[Test build #52924 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52924/consoleFull)**
for PR 11657 at commit
GitHub user srowen opened a pull request:
https://github.com/apache/spark/pull/11657
[SPARK-13823] [CORE] [STREAMING] [SQL] Always specify Charset in String <->
byte[] conversions (and remaining Coverity items)
## What changes were proposed in this pull request?
- Fixes
13 matches
Mail list logo
