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

ISPN-14628 LLEN command #10960

Merged
merged 1 commit into from
May 22, 2023
Merged

ISPN-14628 LLEN command #10960

merged 1 commit into from
May 22, 2023

Conversation

karesti
Copy link
Contributor

@karesti karesti commented May 17, 2023

@karesti karesti requested a review from jabolina May 17, 2023 15:05
@karesti karesti changed the title ISPN-14628 llen command ISPN-14628 LLEN command May 17, 2023
@karesti
Copy link
Contributor Author

karesti commented May 18, 2023

@jabolina rebased from main

Copy link
Member

@jabolina jabolina left a comment

Choose a reason for hiding this comment

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

Looks good, needs another rebase.

@jabolina jabolina merged commit 0ee6a75 into infinispan:main May 22, 2023
3 of 4 checks passed
@jabolina
Copy link
Member

Integrated into main, thanks @karesti!

@karesti karesti deleted the ISPN-14628-LLEN branch May 23, 2023 13:36
Copy link
Member

@wburns wburns left a comment

Choose a reason for hiding this comment

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

Just some minor things

byte[] key = arguments.get(0);
EmbeddedMultimapListCache<byte[], byte[]> listMultimap = handler.getListMultimap();

return CompletionStages.handleAndCompose(listMultimap.size(key) ,(size, t) -> {
Copy link
Member

Choose a reason for hiding this comment

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

Couldn't this just use thenApply on the stage returned from size method?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

you mean not using the handleAndCompose and instead using directly thenApply ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants