Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19635
kindy ping @gatorsmile @hvanhovell
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/19635
The below SQL will throw `AnalysisException`. but it can success on Spark
2.1.x. Hope this can fix soon.
```sql
CREATE TEMPORARY VIEW t4 AS SELECT * FROM VALUES
(CAST(1 AS DOUBLE),
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19635
@gatorsmile @hvanhovell As there were objections on having this in or not,
before resolving the conflict I just wanted to check what do you think about
fixing this behavior. What do you think?
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19635
the tests which are failing are because of the tests added about the
current behavior of Spark (which are introduced looking forward to introduce a
Hive compliant mode).
Therefore, before
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19635
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19635
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86106/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19635
**[Test build #86106 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86106/testReport)**
for PR 19635 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19635
**[Test build #86106 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86106/testReport)**
for PR 19635 at commit
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19635
kindly ping @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19635
@gatorsmile any further comment on this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19635
Hive is interesting. In older versions, it behaves like current Spark. But
in its current master branch the behavior is like after the patch:
```
0: jdbc:hive2://localhost:1> select 'a'
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19635
Oracle behaves like Spark after the patch:
```
select 'a' from dual where 1 in ('01');
// returns 'a'
select 'a' from dual where 1 in (select '01' from dual);
// returns 'a'
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19635
yes, of course. Which ones should I try? Hive and Oracle?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19635
Since we are trying to introduce a new behavior, could you try the other
systems and see how they behave in this scenario?
---
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19635
@gatorsmile I see you point. And of course there is a behavioral change.
For instance, `select 1 in ('01')` before the PR return `false`, after it
returns `true`.
Nonetheless, I think
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19635
I start worrying about the behavior change introduced by this PR.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19635
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/19635
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83445/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19635
**[Test build #83445 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83445/testReport)**
for PR 19635 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19635
**[Test build #83445 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83445/testReport)**
for PR 19635 at commit
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19635
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19635
has anyone any idea of the reason of the failure?
```
java.io.IOException: Failed to delete:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19635
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19635
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83439/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19635
**[Test build #83439 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83439/testReport)**
for PR 19635 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19635
**[Test build #83439 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83439/testReport)**
for PR 19635 at commit
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19635
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/19635
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83436/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19635
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19635
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/19635
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83355/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19635
**[Test build #83355 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83355/testReport)**
for PR 19635 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19635
**[Test build #83355 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83355/testReport)**
for PR 19635 at commit
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19635
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/19635
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83351/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19635
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19635
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83304/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19635
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/19635
**[Test build #83304 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83304/testReport)**
for PR 19635 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19635
**[Test build #83304 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83304/testReport)**
for PR 19635 at commit
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/19635
@mgaido91 Can you update the PR and describe there what you exactly changed?
---
-
To unsubscribe, e-mail:
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/19635
ok to test
---
-
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/19635
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/19635
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
44 matches
Mail list logo