Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22858
Oops, mind fixing PR title too?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22858
@cloud-fan, thanks for doing this backport!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22858
Merged to branch-2.4.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22858
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98151/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22858
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22858
**[Test build #98151 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98151/testReport)**
for PR 22858 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22858
**[Test build #98151 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98151/testReport)**
for PR 22858 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22858
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22858
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22858
Yup, I think strictly we should change. Looks there are two occurrences at
`udf` and `pands_udf` `isinstance(..., str)`.
Another problem at PySpark is, inconsistent type comparison like
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22858
@HyukjinKwon thanks for the information! Shall we replace `str` with
`basestring` in `functions.py` for master branch?
---
-
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22858
Wenchen, this is because
```python
if sys.version >= '3':
basestring = str
```
Is missing. Python 3 does not have `basestring`.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22858
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22858
**[Test build #98126 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98126/testReport)**
for PR 22858 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22858
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98126/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22858
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22858
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22858
BTW the `from_csv` added in 3.0 also use `basestring`, maybe we should
update it as well in master branch.
---
-
To
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22858
**[Test build #98126 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98126/testReport)**
for PR 22858 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22858
cc @HyukjinKwon @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
20 matches
Mail list logo