[GitHub] incubator-predictionio issue #328: [PIO-47] Eliminate enginemanifest for sta...

2016-12-13 Thread pferrel
Github user pferrel commented on the issue: https://github.com/apache/incubator-predictionio/pull/328 The file path is required to be the same on any 2 machines and is explicitly in the metadata. If you have the deploy machine dir in a different path than the train machine dir (actual

[GitHub] incubator-predictionio issue #328: [PIO-47] Eliminate enginemanifest for sta...

2016-12-13 Thread pferrel
Github user pferrel commented on the issue: https://github.com/apache/incubator-predictionio/pull/328 @chanlee514 said: If the absolute filepath is the same in the machines (since SHA-1 hash of filepath is used as engineVersion), there would be no changes in the above

[jira] [Commented] (PIO-47) Remove engine manifest for stateless build

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/PIO-47?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15747066#comment-15747066 ] ASF GitHub Bot commented on PIO-47: --- Github user pferrel commented on the issue: https:

[jira] [Commented] (PIO-47) Remove engine manifest for stateless build

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/PIO-47?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15747013#comment-15747013 ] ASF GitHub Bot commented on PIO-47: --- Github user pferrel commented on the issue: https:

Re: [GitHub] incubator-predictionio issue #328: [PIO-47] Eliminate enginemanifest for sta...

2016-12-13 Thread Chan Lee
If the absolute filepath is the same in the machines (since SHA-1 hash of filepath is used as engineVersion), there would be no changes in the above workflow. You can just copy the engine directory and deploy. If not, you can also provide engine-id, engine-version as CLI arguments. There's some ad