search

numaproj / numaflow-java

Numaflow Java SDK
Apache License 2.0
22 stars 10 forks source link

feat: Add Map batch #132

Closed RohanAshar closed 2 months ago

RohanAshar commented 2 months ago

Fixes https://github.com/numaproj/numaflow/issues/1789 Fixes https://github.com/numaproj/numaflow/issues/1846

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 73.91304% with 48 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@3c2ec6f). Learn more about missing BASE report.

Files Patch % Lines
...java/io/numaproj/numaflow/batchmapper/Service.java 74.35% 15 Missing and 5 partials :warning:
.../java/io/numaproj/numaflow/batchmapper/Server.java 57.77% 16 Missing and 3 partials :warning:
...java/io/numaproj/numaflow/batchmapper/Message.java 70.00% 3 Missing :warning:
...maproj/numaflow/batchmapper/DatumIteratorImpl.java 87.50% 1 Missing and 1 partial :warning:
...a/io/numaproj/numaflow/shared/GrpcServerUtils.java 50.00% 1 Missing and 1 partial :warning:
...va/io/numaproj/numaflow/batchmapper/Constants.java 0.00% 1 Missing :warning:
...a/io/numaproj/numaflow/batchmapper/GRPCConfig.java 80.00% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #132 +/- ## ======================================= Coverage ? 59.43% Complexity ? 361 ======================================= Files ? 120 Lines ? 2384 Branches ? 169 ======================================= Hits ? 1417 Misses ? 837 Partials ? 130 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

KeranYang commented 2 months ago

@RohanAshar I have manually created the repository https://quay.io/repository/numaio/numaflow-java/batch-map-example and granted the CI account permission to push the image. Should be good to monitor the build-push workflow after merging.