Github user tdas commented on a diff in the pull request:

    https://github.com/apache/spark/pull/5428#discussion_r28384609
  
    --- Diff: 
streaming/src/main/scala/org/apache/spark/streaming/StreamingContext.scala ---
    @@ -114,11 +123,15 @@ class StreamingContext private[streaming] (
     
       private[streaming] val isCheckpointPresent = (cp_ != null)
     
    +  private[streaming] val isSparkContextPresent = (sc_ != null)
    --- End diff --
    
    Good point. My idea was to keep it consistent with isCheckpointPresent, 
which has the semantics of "was checkpoint present at the time of creation of 
the StremaingContext". So this was meant to be "was an existing sparkcontext 
used to create the stremaingContext". But i can see that it can be confusing. 
Will remove that completely.


---
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to