Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15513
How about "a string literal" vs "a string expression"?
---
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15513
Also, will try to consolidate multiple usages and take out `_FUNC_` in
extended part too.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15513
@rxin, How about this?
**Literal only**
```
a string type value.
a numeric type value that defines ...
a binary type value that defines ... For example, ...
```
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67314/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67314 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67314/consoleFull)**
for PR 15513 at commit
[`7841860`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67315/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67315 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67315/consoleFull)**
for PR 15513 at commit
[`01eecfe`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67313/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67313 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67313/consoleFull)**
for PR 15513 at commit
[`91d2ab5`](https://github.com/apache/spark/commit/
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15513
@rxin Sure, let me try to sweep it and make this simpler and more correct
(including the comments above). Thanks for your quick response!
---
If your project is set up for it, you can reply to
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15513
@HyukjinKwon I'd try to keep the documentation more concise. For example
"any numeric type or any nonnumeric type expression that can be implicitly
converted to numeric type" is very long. Just say "a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67315 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67315/consoleFull)**
for PR 15513 at commit
[`01eecfe`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67314 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67314/consoleFull)**
for PR 15513 at commit
[`7841860`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67313 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67313/consoleFull)**
for PR 15513 at commit
[`91d2ab5`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67275/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
Merged build finished. Test FAILed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67275 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67275/consoleFull)**
for PR 15513 at commit
[`d927bff`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67274/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
Merged build finished. Test FAILed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67274 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67274/consoleFull)**
for PR 15513 at commit
[`73ccda0`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67275 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67275/consoleFull)**
for PR 15513 at commit
[`d927bff`](https://github.com/apache/spark/commit/d
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15513
@rxin and @srowen I will double-check this tomorrow again but I think it
looks ready to be reviewed. I appreciate if you both take a look please.
---
If your project is set up for it, you can r
101 - 125 of 125 matches
Mail list logo