Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pipe: Move parsing event logic into PipeProcessorSubtask to separate CPU intensive and IO intensive tasks #11965

Merged
merged 17 commits into from
Feb 7, 2024

Conversation

DanielWang2035
Copy link
Collaborator

Changes:

  • moved parse event logic from connector plugins to PipeProcessorSubtask.
  • added a buffer queue in PipeProcessorSubtask to store parsed events.

Note:

  • now the sequence of an event is: extractor -> parse -> processor -> connector
  • now commit-ids are assigned to parsed events, not the ones before parsing

For memory control, parsed events are stored in two places: PipeProcessorSubtask#parsedEventQueue and PipeEventCollector#bufferQueue. The former one will only store events parsed from one source event, and PipeProcessorSubtask will not parse new events if the latter one is not empty. So there won't be too much parsed events in queue..

@SteveYurongSu SteveYurongSu self-requested a review January 24, 2024 07:56
@SteveYurongSu SteveYurongSu self-assigned this Jan 24, 2024
# Conflicts:
#	iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/task/subtask/processor/PipeProcessorSubtask.java
Copy link
Collaborator

@Caideyipi Caideyipi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PTAL~

@SteveYurongSu SteveYurongSu changed the title Pipe: parse events moved to PipeProcessorSubtask Pipe: Move parsing event logic into PipeProcessorSubtask to separate CPU intensive and IO intensive tasks Feb 7, 2024
@SteveYurongSu SteveYurongSu merged commit 1e37acc into apache:master Feb 7, 2024
36 checks passed
SzyWilliam pushed a commit to SzyWilliam/iotdb that referenced this pull request Nov 26, 2024
…CPU intensive and IO intensive tasks (apache#11965)

Co-authored-by: Caideyipi <87789683+Caideyipi@users.noreply.github.com>
Co-authored-by: Steve Yurong Su <rong@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants