Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22458
Hi, @zheyuan28 . Thank you for your first contribution. You've finished
almost things. So, in order to be able to merge, we need the followings.
- Could you merge the Hive test case PR,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22458
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/22458
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96411/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22458
**[Test build #96411 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96411/testReport)**
for PR 22458 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22458
**[Test build #96411 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96411/testReport)**
for PR 22458 at commit
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22458
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/22458
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96399/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22458
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/22458
**[Test build #96399 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96399/testReport)**
for PR 22458 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22458
**[Test build #96399 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96399/testReport)**
for PR 22458 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22458
LGTM. If there are more properties like `originalViewText` which are
useless to Spark and only need to be displayed, I'd suggest we create a map for
them, instead of adding more and more fields
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22458
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96371/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22458
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/22458
**[Test build #96371 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96371/testReport)**
for PR 22458 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22458
**[Test build #96371 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96371/testReport)**
for PR 22458 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22458
This is reasonable to me. cc @jiangxb1987 @cloud-fan @hvanhovell
---
-
To unsubscribe, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22458
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user zheyuan28 commented on the issue:
https://github.com/apache/spark/pull/22458
cc @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
18 matches
Mail list logo