Skip to content
This repository has been archived by the owner on Feb 15, 2022. It is now read-only.

Initial commit of service map stateful processor plugin #28

Merged
merged 7 commits into from
Sep 22, 2020

Conversation

AustinTag
Copy link
Contributor

Issue #, if available:

Description of changes:
Adding initial commit of stateful service map processor plugin

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Copy link
Contributor

@kowshikn kowshikn left a comment

Choose a reason for hiding this comment

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

This looks good for the first revision. Just one thought,

            //TODO: Look into faster way to move cursor up N elements
            for(long i=0; i<start; i++) {
                cursor.next();
            }

If we can not optimize this we should do the whole scan and find keyranges once rather than each processor's iterate doing it.

@erosas erosas closed this Sep 22, 2020
@erosas erosas reopened this Sep 22, 2020
@AustinTag AustinTag merged commit 20047d5 into master Sep 22, 2020
@kowshikn kowshikn deleted the service-map-stateful branch November 16, 2020 22:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants