Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19124
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19124
I created SPARK-21929 for **"Support `ALTER TABLE table_name ADD
COLUMNS(..)` for ORC data source"**.
For Parquet ALTER TABLE, yes. I think I can include that here.
But, for the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19124
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81432/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19124
**[Test build #81432 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81432/testReport)**
for PR 19124 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19124
Could this PR cover this scenario?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19124
For that, no. It's not considered yet like the other code path.
---
-
To unsubscribe, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19124
Altering table add column with illegal column names will issue an error
message?
---
-
To unsubscribe, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19124
Parquet works. I tested.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19124
How about Parquet?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19124
Ah, there are too many missing things in ORC code path.
`AlterTableAddColumnsCommand` seems not to allow ORC in
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19124
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/19124
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81430/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19124
**[Test build #81430 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81430/testReport)**
for PR 19124 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19124
Please also fix `AlterTableAddColumnsCommand`?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19124
**[Test build #81432 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81432/testReport)**
for PR 19124 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19124
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/19124
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81424/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19124
**[Test build #81424 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81424/testReport)**
for PR 19124 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19124
**[Test build #81430 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81430/testReport)**
for PR 19124 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19124
That is normal. When we find a bug, it normally means we ignore it in more
than one place. Thus, we need to check all the other code paths that could
trigger it.
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19124
Thank you, @gatorsmile . The PR becomes much general.
When I started this PR, I didn't notice that the current Spark had so many
missing cases like this.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19124
**[Test build #81424 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81424/testReport)**
for PR 19124 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19124
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81408/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19124
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/19124
**[Test build #81408 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81408/testReport)**
for PR 19124 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19124
**[Test build #81408 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81408/testReport)**
for PR 19124 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19124
I just updated the output answer file to show the result for review only.
---
-
To unsubscribe, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19124
Hi, @gatorsmile .
I can fix it in most cases, but we have the [following test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19124
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/19124
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81405/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19124
**[Test build #81405 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81405/testReport)**
for PR 19124 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19124
**[Test build #81405 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81405/testReport)**
for PR 19124 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19124
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/19124
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81404/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19124
**[Test build #81404 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81404/testReport)**
for PR 19124 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19124
**[Test build #81404 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81404/testReport)**
for PR 19124 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19124
**[Test build #81403 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81403/testReport)**
for PR 19124 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19124
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81403/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19124
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/19124
**[Test build #81403 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81403/testReport)**
for PR 19124 at commit
40 matches
Mail list logo