Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20168
Overall looks good to me. Just some minor comments regarding with code
comments and naming.
---
-
To unsubscribe, e-mail: reviews
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86156/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #86156 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86156/testReport)**
for PR 20168 at commit
[`896ccc2`](https://github.com/apache/spark/commit/8
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20168
Btw, I think this isn't only to add non-integer image formats. So the PR
title may be changed too. Like "Add ImageSchema support for all OpenCV image
types"?
---
--
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #86156 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86156/testReport)**
for PR 20168 at commit
[`896ccc2`](https://github.com/apache/spark/commit/89
Github user imatiach-msft commented on the issue:
https://github.com/apache/spark/pull/20168
@MrBago @tomasatdatabricks the changes look good to me, I went through
everything one more time, I'll sign off as soon as the python tests are fixed
(it looks like there were some style issues
Github user imatiach-msft commented on the issue:
https://github.com/apache/spark/pull/20168
@MrBago @tomasatdatabricks I think the breaking changes are fine, the code
was marked experimental and it is expected that the interfaces will change a
lot initially based on early feedback.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86149/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #86149 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86149/testReport)**
for PR 20168 at commit
[`9ec8cd3`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #86149 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86149/testReport)**
for PR 20168 at commit
[`9ec8cd3`](https://github.com/apache/spark/commit/9e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86148/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #86148 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86148/testReport)**
for PR 20168 at commit
[`68a5a94`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #86148 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86148/testReport)**
for PR 20168 at commit
[`68a5a94`](https://github.com/apache/spark/commit/68
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86061/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #86061 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86061/testReport)**
for PR 20168 at commit
[`d2a864e`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #86061 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86061/testReport)**
for PR 20168 at commit
[`d2a864e`](https://github.com/apache/spark/commit/d2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86059/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #86059 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86059/testReport)**
for PR 20168 at commit
[`2401add`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #86059 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86059/testReport)**
for PR 20168 at commit
[`2401add`](https://github.com/apache/spark/commit/24
Github user tomasatdatabricks commented on the issue:
https://github.com/apache/spark/pull/20168
@MrBago Here is the description of the breaking changes.
ImageSchema.ocvTypes and ImageSchema.javaOcvTypes changed types from
Map[String,Int] to list of OpenCvType.
ImageSchema.oc
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85884/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #85884 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85884/testReport)**
for PR 20168 at commit
[`763c8a6`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #85884 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85884/testReport)**
for PR 20168 at commit
[`763c8a6`](https://github.com/apache/spark/commit/76
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85880/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #85880 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85880/testReport)**
for PR 20168 at commit
[`48eddf1`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #85880 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85880/testReport)**
for PR 20168 at commit
[`48eddf1`](https://github.com/apache/spark/commit/48
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85878/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20168
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #85878 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85878/testReport)**
for PR 20168 at commit
[`eee25ce`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20168
**[Test build #85878 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85878/testReport)**
for PR 20168 at commit
[`eee25ce`](https://github.com/apache/spark/commit/ee
37 matches
Mail list logo
