Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/9207
Closing this in favor of the approach taken in
https://github.com/apache/spark/pull/19876
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82102/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #82102 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82102/consoleFull)**
for PR 9207 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #82102 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82102/consoleFull)**
for PR 9207 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/9207
@MLNick: Do you have the bandwith to revisit this? I'm open to refactoring
to a more plug-gable approach if we've got the review bandwidth for it.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68724/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #68724 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68724/consoleFull)**
for PR 9207 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #68724 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68724/consoleFull)**
for PR 9207 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/9207
Ping @jkbradley / @mlnick - we are coming up on a year since we told people
we would add PMML export to Spark in about a year so I figured I'd try and
update this again. I'll update this PR tonight
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65123/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #65123 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65123/consoleFull)**
for PR 9207 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #65123 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65123/consoleFull)**
for PR 9207 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/9207
ping @MLnick now that 2.0 is out for awhile.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63295/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #63295 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63295/consoleFull)**
for PR 9207 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #63295 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63295/consoleFull)**
for PR 9207 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63193/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #63193 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63193/consoleFull)**
for PR 9207 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #63193 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63193/consoleFull)**
for PR 9207 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63086/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #63086 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63086/consoleFull)**
for PR 9207 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #63086 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63086/consoleFull)**
for PR 9207 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/9207
Now that 2.0 RC5 has passed - maybe it would be an OK time to revisit this?
I think it can also be an ok base for adding more formats if we need to.
---
If your project is set up for it, you can
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62478/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #62478 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62478/consoleFull)**
for PR 9207 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #62478 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62478/consoleFull)**
for PR 9207 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61799/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #61799 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61799/consoleFull)**
for PR 9207 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #61799 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61799/consoleFull)**
for PR 9207 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/9207
HiveSubmit failures seem unrelated, jenkins retest this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61784/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #61784 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61784/consoleFull)**
for PR 9207 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #61784 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61784/consoleFull)**
for PR 9207 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #61684 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61684/consoleFull)**
for PR 9207 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61684/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #61684 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61684/consoleFull)**
for PR 9207 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61683/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #61683 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61683/consoleFull)**
for PR 9207 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #61683 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61683/consoleFull)**
for PR 9207 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/9207
@MLnick I like that idea! I've done the shortcut approach on the root
(since thats the shortcut we already had) - but I'll look at adding the
shortcut on the MLWriter its self as well. Thanks!
---
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/9207
@holden that's true for the fully generic approach. But `DataFrameWriter`
for example exposes `json` as a shortcut (type-safe in a way) for
`format("json")`.
I think we can achieve something
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/9207
@mengxr One downside of going the generic writer way is we loose compile
time checking as to if the requested format is supported.
---
If your project is set up for it, you can reply to this email
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60976/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9207
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #60976 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60976/consoleFull)**
for PR 9207 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9207
**[Test build #60976 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60976/consoleFull)**
for PR 9207 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/9207
So I guess I'm wondering what our plans for PMML look like - I'm happy to
update this or go in the direction @MLnick suggested if thats what we want?
---
If your project is set up for it, you can
58 matches
Mail list logo