Skip to content

[docs] Document how to optimize multihost performance for unbalanced large data. - #6758

Merged
blp merged 1 commit into
mainfrom
multihost-connectors
Jul 30, 2026
Merged

[docs] Document how to optimize multihost performance for unbalanced large data.#6758
blp merged 1 commit into
mainfrom
multihost-connectors

Conversation

@blp

@blp blp commented Jul 29, 2026

Copy link
Copy Markdown
Member

No description provided.

@blp
blp requested a review from ryzhyk July 29, 2026 22:49
@blp blp self-assigned this Jul 29, 2026
@blp blp added documentation Improvements or additions to documentation connectors Issues related to the adapters/connectors crate multihost Related to multihost or distributed pipelines labels Jul 29, 2026
If the pipeline is stopped and restarted during a retry, the connector resumes from the last successfully
ingested table version. This guarantees that no data loss occurs due to object store read errors.

## Optimizing multihost performance for unbalanced large data

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

much of this is not specific to Delta, maybe there should be a separate page about this.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is definitely true as the amount of advice increases; I wasn't sure whether it was yet for n = 2.

If the pipeline is stopped and restarted during a retry, the connector resumes from the last successfully
ingested table version. This guarantees that no data loss occurs due to object store read errors.

## Optimizing multihost performance for unbalanced large data

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This advice doesn't work for tables with lateness, since different connectors can go out of sync.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I fixed this by adding a warning paragraph:

> ⚠️ [Tables with LATENESS] cannot correctly be divided into multiple
> connectors this way, because the different connectors can read data
> "out of sync" from another.

[Tables with LATENESS]: /sql/streaming/#lateness-expressions

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

perfect

be added to the partition whose events have been completely
processed.

## Optimizing multihost performance for unbalanced large data

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Kafka already doesn't guarantee lateness across multiple partitions, but we have the synchronize_partitions feature, which is not applicable if we split ingest across multiple connectors.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I fixed this by adding a warning paragraph:

> ⚠️ The [synchronize_partitions] feature does not work across
> connectors, only within a single connector.  Therefore, [tables with
> LATENESS] cannot correctly be divided into multiple connectors this
> way, because the different connectors can read data "out of sync"
> from another.

[tables with LATENESS]: /sql/streaming/#lateness-expressions
[synchronize_partitions]: #synchronize_partitions

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

sounds good

@mythical-fred mythical-fred left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Useful docs addition — one typo to fix (appears in both files).

Comment thread docs.feldera.com/docs/connectors/sources/delta.md
Comment thread docs.feldera.com/docs/connectors/sources/kafka.md
@blp
blp force-pushed the multihost-connectors branch from b5858de to 09eeeef Compare July 30, 2026 17:08
…data.

Signed-off-by: Ben Pfaff <blp@feldera.com>
@blp
blp force-pushed the multihost-connectors branch from 09eeeef to 10ddb18 Compare July 30, 2026 18:00
@blp
blp added this pull request to the merge queue Jul 30, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to invalid changes in the merge commit Jul 30, 2026
@blp
blp enabled auto-merge July 30, 2026 18:03
@blp
blp added this pull request to the merge queue Jul 30, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 30, 2026
@blp
blp added this pull request to the merge queue Jul 30, 2026

@mythical-fred mythical-fred left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Re-approving after rebase + follow-up. The new warning callouts about LATENESS + synchronize_partitions not working across connectors are exactly the caveat this section needed. Nice.

Merged via the queue into main with commit e1a3f0c Jul 30, 2026
1 check passed
@blp
blp deleted the multihost-connectors branch July 30, 2026 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

connectors Issues related to the adapters/connectors crate documentation Improvements or additions to documentation multihost Related to multihost or distributed pipelines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants