Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Aug 19, 2015

This PR includes the following fixes:

  1. Use range instead of xrange in queue_stream.py to support Python 3.
  2. Fix the issue that utf8_decoder will return bytes rather than str when receiving an empty bytes in Python 3.
  3. Fix the commands in docs so that the user can copy them directly to the command line. The previous commands was broken in the middle of a path, so when copying to the command line, the path would be split to two parts by the extra spaces, which forces the user to fix it manually.

@SparkQA
Copy link

SparkQA commented Aug 19, 2015

Test build #41255 has finished for PR 8315 at commit 369ac49.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@tdas
Copy link
Contributor

tdas commented Aug 20, 2015

Thanks! merging this to master and 1.5!

asfgit pushed a commit that referenced this pull request Aug 20, 2015
…Streaming and some docs

This PR includes the following fixes:
1. Use `range` instead of `xrange` in `queue_stream.py` to support Python 3.
2. Fix the issue that `utf8_decoder` will return `bytes` rather than `str` when receiving an empty `bytes` in Python 3.
3. Fix the commands in docs so that the user can copy them directly to the command line. The previous commands was broken in the middle of a path, so when copying to the command line, the path would be split to two parts by the extra spaces, which forces the user to fix it manually.

Author: zsxwing <zsxwing@gmail.com>

Closes #8315 from zsxwing/SPARK-9812.

(cherry picked from commit 1f29d50)
Signed-off-by: Tathagata Das <tathagata.das1565@gmail.com>
@asfgit asfgit closed this in 1f29d50 Aug 20, 2015
@zsxwing zsxwing deleted the SPARK-9812 branch August 20, 2015 05:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants