Github user shimamoto commented on the issue:

    https://github.com/apache/incubator-predictionio/pull/441
  
    I also agree with takezoe!
    
    @mars `BatchPredict` seems to make multiple SparkContexts. The 
`WorkflowContext.apply` method creates a new instance of the SparkContext class.
    
https://github.com/apache/incubator-predictionio/blob/v0.12.0-incubating/core/src/main/scala/org/apache/predictionio/workflow/BatchPredict.scala#L160
    
https://github.com/apache/incubator-predictionio/blob/v0.12.0-incubating/core/src/main/scala/org/apache/predictionio/workflow/BatchPredict.scala#L183
    Is this supposed to be this way?


---

Reply via email to