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.

Sign up for GitHub

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

Jump to bottom

Implement message queue of actor model #62

Closed
Keith-CY opened this issue Dec 7, 2022 · 2 comments
Closed

Implement message queue of actor model #62

Keith-CY opened this issue Dec 7, 2022 · 2 comments
Assignees
@Keith-CY
Labels
enhancement New feature or request

Comments

@Keith-CY
Copy link
Member

Keith-CY commented Dec 7, 2022

The message queue in #29 is roughly implemented with a Map for fast delivery so other modules won't be blocked.

It's expected to be done with redis stream as designed in #17
@Keith-CY Keith-CY added the enhancement New feature or request label Dec 7, 2022
@Keith-CY Keith-CY self-assigned this Dec 7, 2022
@Keith-CY
Copy link
Member Author

Keith-CY commented Dec 7, 2022

It was expected to be done by 12/08, but there's no bandwidth for it. I'll try to propose a PR by 12/12

@Keith-CY Keith-CY moved this to 🔖 Ready in CKB JS Backlog Dec 7, 2022
Keith-CY added a commit that referenced this issue Dec 12, 2022
@Keith-CY
feat: implement mailbox with redis stream
37c7eb6 
This PR implements the message queue of actor model with redis stream

Ref: #62
Keith-CY added a commit that referenced this issue Dec 12, 2022
@Keith-CY
feat: implement mailbox with redis stream
837a4a4 
This PR implements the mailbox of actor model with redis stream

Ref: #62
@Keith-CY Keith-CY mentioned this issue Dec 12, 2022
Merged
@Keith-CY
Copy link
Member Author

PR: #74

@Keith-CY Keith-CY moved this from 🔖 Ready to 👀 In review in CKB JS Backlog Dec 12, 2022
Keith-CY added a commit that referenced this issue Dec 12, 2022
@Keith-CY
feat: implement mailbox with redis stream
f0e5c0d 
This PR implements the mailbox of actor model with redis stream

Ref: #62
Keith-CY added a commit that referenced this issue Dec 12, 2022
@Keith-CY
feat: implement mailbox with redis stream
efa6d47 
This PR implements the mailbox of actor model with redis stream

Ref: #62
Keith-CY added a commit that referenced this issue Dec 12, 2022
@Keith-CY
feat: implement mailbox with redis stream
ac49039 
This PR implements the mailbox of actor model with redis stream

Ref: #62
Keith-CY added a commit that referenced this issue Dec 12, 2022
@Keith-CY
feat: implement mailbox with redis stream
730f362 
This PR implements the mailbox of actor model with redis stream

Ref: #62
@Keith-CY Keith-CY mentioned this issue Dec 15, 2022
Closed
3 tasks
@Keith-CY Keith-CY moved this from 👀 In review to ✅ Done in CKB JS Backlog Dec 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Keith-CY Keith-CY

Labels
enhancement New feature or request
Projects
CKB JS Backlog
Archived in project
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Keith-CY