You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Ufuk Celebi (JIRA)" <ji...@apache.org> on 2017/09/21 19:34:00 UTC

[jira] [Created] (FLINK-7665) Use wait/notify in ContinuousFileReaderOperator

Ufuk Celebi created FLINK-7665:
----------------------------------

             Summary: Use wait/notify in ContinuousFileReaderOperator
                 Key: FLINK-7665
                 URL: https://issues.apache.org/jira/browse/FLINK-7665
             Project: Flink
          Issue Type: Improvement
          Components: Streaming Connectors
    Affects Versions: 1.4.0
            Reporter: Ufuk Celebi
            Priority: Minor


{{ContinuousFileReaderOperator}} has the following loop to receive input splits:
{code}
synchronized (checkpointLock) {
  if (currentSplit == null) {
    currentSplit = this.pendingSplits.poll();
    if (currentSplit == null) {
      if (this.shouldClose) {
        isRunning = false;
      } else {
        checkpointLock.wait(50);
      }
      continue;
    }
  }
}
{code}

I think we can replace this with a {{wait()}} and {{notify()}} in {{addSplit}} and {{close}}.

If there is a reason to keep the {{wait(50)}}, feel free to close this issue. :-)



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)