[ 
https://issues.apache.org/jira/browse/SPARK-26300?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16716913#comment-16716913
 ] 

ASF GitHub Bot commented on SPARK-26300:
----------------------------------------

SparkQA commented on issue #23251: [SPARK-26300][SS] Remove a redundant 
`checkForStreaming` call
URL: https://github.com/apache/spark/pull/23251#issuecomment-446176845
 
 
   **[Test build #99961 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99961/testReport)**
 for PR 23251 at commit 
[`b1e71ee`](https://github.com/apache/spark/commit/b1e71ee7a723d63f1cf3c0754f2372eb185439d3).
    * This patch passes all tests.
    * This patch merges cleanly.
    * This patch adds no public classes.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> The `checkForStreaming`  mothod  may be called twice in `createQuery`
> ---------------------------------------------------------------------
>
>                 Key: SPARK-26300
>                 URL: https://issues.apache.org/jira/browse/SPARK-26300
>             Project: Spark
>          Issue Type: Improvement
>          Components: Structured Streaming
>    Affects Versions: 2.4.0
>            Reporter: liuxian
>            Priority: Minor
>
> If {{checkForContinuous}} is called ( {{checkForStreaming}} is called in 
> {{checkForContinuous}} ), the {{checkForStreaming}} mothod will be called 
> twice in {{createQuery}} , this is not necessary, and the 
> {{checkForStreaming}} method has a lot of statements, so it's better to 
> remove one of them.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org

Reply via email to