Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce a warning during the serialization of MOJO model #2921

Closed
exalate-issue-sync bot opened this issue May 22, 2023 · 5 comments
Closed

Introduce a warning during the serialization of MOJO model #2921

exalate-issue-sync bot opened this issue May 22, 2023 · 5 comments
Assignees

Comments

@exalate-issue-sync
Copy link

For details, refer to support ticket: https://support.h2o.ai/a/tickets/100534

@exalate-issue-sync
Copy link
Author

Marek Novotny commented: Alternative solution is to throw exception if H2OMOJOModel is being serialized:

H2OMOJOModel is currently Java serializable and serves as a closure for Spark UDFs that are responsible for model scoring. However, H2OMOJOModel which has been serialized, moved to different Spark application and deserialized can't be used for scoring. Such a H2OMOJOModel will get disconnected from H2O-3 mojo artifacts that were located on Spark executors of the original cluster.

So the solution would be to create a class that will play role of a closure for Spark UDFs and make H2OMOJOModel unserializable.

@exalate-issue-sync
Copy link
Author

Arun Aryasomayajula commented: Any updates on this Jira Marek?

@exalate-issue-sync
Copy link
Author

Marek Novotny commented: [~accountid:5fa438f822f3990076aa232d] I’ve created the PR [https://github.com//pull/2742|https://github.com//pull/2742|smart-link] to cover that.

@DinukaH2O
Copy link

JIRA Issue Migration Info

Jira Issue: SW-2655
Assignee: Marek Novotny
Reporter: Neema Mashayekhi
State: Resolved
Fix Version: 3.36.0.4-1
Attachments: N/A
Development PRs: Available

Linked PRs from JIRA

#2742

@hasithjp
Copy link
Member

JIRA Issue Migration Info Cont'd

Jira Issue Created Date: 2021-12-01T14:55:33.561-0800

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants