Uploaded image for project: 'Flink'
  1. Flink
  2. FLINK-7665

Use wait/notify in ContinuousFileReaderOperator

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Reopened
    • Not a Priority
    • Resolution: Unresolved
    • 1.4.0
    • None
    • Connectors / Common
    • None

    Description

      ContinuousFileReaderOperator has the following loop to receive input splits:

      synchronized (checkpointLock) {
        if (currentSplit == null) {
          currentSplit = this.pendingSplits.poll();
          if (currentSplit == null) {
            if (this.shouldClose) {
              isRunning = false;
            } else {
              checkpointLock.wait(50);
            }
            continue;
          }
        }
      }
      

      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.

      Attachments

        Activity

          People

            Unassigned Unassigned
            uce Ufuk Celebi
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated: