Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22788
**[Test build #98365 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98365/testReport)**
for PR 22788 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22788
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22788
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/22788
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98270/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22788
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/22788
**[Test build #98270 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98270/testReport)**
for PR 22788 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22788
**[Test build #98270 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98270/testReport)**
for PR 22788 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22788
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/22788
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22788
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98003/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22788
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/22788
**[Test build #98003 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98003/testReport)**
for PR 22788 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22788
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22788
**[Test build #98003 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98003/testReport)**
for PR 22788 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22788
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22788
@dongjoon-hyun Oh I see. The ambiguousness is in the results of `sql` for
several inputs.
---
-
To unsubscribe, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22788
I agree with the problem described in the PR description that
`UnresolvedAttribute.sql` is not ideal. But we should just update
`UnresolvedAttribute.sql`, not the `name` method. `name` is used in
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22788
@viirya . Please see the all four examples. I guess you missed the context.
- https://github.com/apache/spark/pull/22788#pullrequestreview-166797320
BTW, I'm good for any methods
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22788
From above examples,
```scala
scala>
$"`a.b`".expr.asInstanceOf[org.apache.spark.sql.catalyst.analysis.UnresolvedAttribute].sql
res1: String = `a.b` // ambiguous
```
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22788
yea, only use json if it's a nested column.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/22788
@cloud-fan I like the idea of using JSON, but that will also change the
definition of string format. Do we just use JSON for nested case so the
existing data source doesn't have to be changed?
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22788
Yea I think so, we can even use JSON to be safer. e.g. for `a.b.c.d`, we
can encode it as a json array [a,b,c,d]. At data source side, use a json parser
to read it back.
---
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/22788
@cloud-fan @dongjoon-hyun instead of changing `Filter` API, do you think
using proper escaped char like this PR in
https://github.com/apache/spark/pull/22573 is a good approach?
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22788
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/22788
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97881/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22788
**[Test build #97881 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97881/testReport)**
for PR 22788 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22788
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22788
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/22788
**[Test build #97881 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97881/testReport)**
for PR 22788 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22788
Retest this please.
---
-
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/22788
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22788
**[Test build #97877 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97877/testReport)**
for PR 22788 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22788
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97877/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22788
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22788
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/22788
**[Test build #97877 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97877/testReport)**
for PR 22788 at commit
36 matches
Mail list logo
