Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Jun 29, 2016

What changes were proposed in this pull request?

Fixed the following error:

>>> sqlContext.readStream
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "...", line 442, in readStream
    return DataStreamReader(self._wrapped)
NameError: global name 'DataStreamReader' is not defined

How was this patch tested?

The added test.

@zsxwing zsxwing changed the title SQLContext should import DataStreamReader [SPARK-16268][PySpark]SQLContext should import DataStreamReader Jun 29, 2016
@zsxwing
Copy link
Member Author

zsxwing commented Jun 29, 2016

cc @tdas @brkyvz

@brkyvz
Copy link
Contributor

brkyvz commented Jun 29, 2016

LGTM pending tests

@SparkQA
Copy link

SparkQA commented Jun 29, 2016

Test build #61424 has finished for PR 13958 at commit 0e2c3d8.

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

@tdas
Copy link
Contributor

tdas commented Jun 29, 2016

My patch #13955 has the partial fix, but I realized that there are no tests so my patch did not actually fix this. Let merge this and then update my patch. Merging this to master and 2.0.

asfgit pushed a commit that referenced this pull request Jun 29, 2016
## What changes were proposed in this pull request?

Fixed the following error:
```
>>> sqlContext.readStream
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "...", line 442, in readStream
    return DataStreamReader(self._wrapped)
NameError: global name 'DataStreamReader' is not defined
```

## How was this patch tested?

The added test.

Author: Shixiong Zhu <shixiong@databricks.com>

Closes #13958 from zsxwing/fix-import.

(cherry picked from commit 5bf8881)
Signed-off-by: Tathagata Das <tathagata.das1565@gmail.com>
@asfgit asfgit closed this in 5bf8881 Jun 29, 2016
@zsxwing zsxwing deleted the fix-import branch June 29, 2016 02:17
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.

4 participants