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

chore: Batch ack requests and sink responses for better performance #149

Merged
merged 1 commit into from
Oct 31, 2024

batch ack requests and sink respones

4583ddc
Select commit
Loading
Failed to load commit list.
Merged

chore: Batch ack requests and sink responses for better performance #149

batch ack requests and sink respones
4583ddc
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Oct 30, 2024 in 0s

No report found to compare against

View this Pull Request on Codecov

No report found to compare against

Annotations

Check warning on line 208 in src/main/java/io/numaproj/numaflow/sinker/SinkerTestKit.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/io/numaproj/numaflow/sinker/SinkerTestKit.java#L207-L208

Added lines #L207 - L208 were not covered by tests

Check warning on line 211 in src/main/java/io/numaproj/numaflow/sinker/SinkerTestKit.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/io/numaproj/numaflow/sinker/SinkerTestKit.java#L210-L211

Added lines #L210 - L211 were not covered by tests

Check warning on line 214 in src/main/java/io/numaproj/numaflow/sinker/SinkerTestKit.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/io/numaproj/numaflow/sinker/SinkerTestKit.java#L213-L214

Added lines #L213 - L214 were not covered by tests

Check warning on line 177 in src/main/java/io/numaproj/numaflow/sourcer/SourcerTestKit.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/io/numaproj/numaflow/sourcer/SourcerTestKit.java#L172-L177

Added lines #L172 - L177 were not covered by tests