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

Add filter argument to ringbuffer readMany() method and a test for it API-1294 #1414

Merged
merged 2 commits into from
Nov 20, 2022

Conversation

fatihozer0
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Nov 5, 2022

Codecov Report

Merging #1414 (511a93f) into master (8564b65) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1414      +/-   ##
==========================================
- Coverage   93.28%   93.27%   -0.01%     
==========================================
  Files         463      464       +1     
  Lines       16347    16386      +39     
  Branches     1329     1333       +4     
==========================================
+ Hits        15249    15284      +35     
- Misses        801      803       +2     
- Partials      297      299       +2     
Impacted Files Coverage Δ
src/nearcache/NearCache.ts 92.24% <0.00%> (-1.56%) ⬇️
src/proxy/NearCachedMapProxy.ts 93.05% <0.00%> (-1.28%) ⬇️
src/invocation/InvocationService.ts 94.11% <0.00%> (-1.18%) ⬇️
src/core/HazelcastError.ts 77.27% <0.00%> (-0.76%) ⬇️
src/protocol/ErrorFactory.ts 64.66% <0.00%> (-0.76%) ⬇️
src/HazelcastClient.ts 93.37% <0.00%> (-0.04%) ⬇️
src/network/Connection.ts 94.22% <0.00%> (ø)
src/proxy/ringbuffer/LazyReadResultSet.ts 100.00% <0.00%> (ø)
...c/serialization/compact/CompactStreamSerializer.ts 100.00% <0.00%> (ø)
src/codec/MapRemoveAllCodec.ts 100.00% <0.00%> (ø)
... and 3 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@srknzl srknzl changed the title Add filter argument to ringbuffer readMany() method and a test for it Add filter argument to ringbuffer readMany() method and a test for it API-1294 Nov 7, 2022
src/proxy/Ringbuffer.ts Outdated Show resolved Hide resolved
src/proxy/Ringbuffer.ts Outdated Show resolved Hide resolved
src/proxy/Ringbuffer.ts Outdated Show resolved Hide resolved
@fatihozer0 fatihozer0 merged commit 18c7f58 into hazelcast:master Nov 20, 2022
harunalpak pushed a commit to harunalpak/hazelcast-nodejs-client that referenced this pull request Nov 23, 2022
… API-1294 (hazelcast#1414)

* add filter argument to ringbuffer readMany() method and a test for it

* fix some typos
@fatihozer0 fatihozer0 deleted the add_filter_argument branch November 26, 2022 13:05
@fatihozer0 fatihozer0 modified the milestones: 5.2.0, Backlog Nov 26, 2022
@degerhz degerhz modified the milestones: Backlog, 5.2.0 Nov 26, 2022
harunalpak pushed a commit to harunalpak/hazelcast-nodejs-client that referenced this pull request Dec 8, 2022
… API-1294 (hazelcast#1414)

* add filter argument to ringbuffer readMany() method and a test for it

* fix some typos
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants