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

MoveNoteProcessor in its own thread #30

Merged
merged 1 commit into from
Aug 2, 2023

Conversation

ariard
Copy link
Contributor

@ariard ariard commented Jul 28, 2023

Building on top of the SQLite support #24, this PR moves NoteProcessor out of ServiceManager and aims to be the module receiving the DB write/read request from the other components (client handler, gateway, RPC client).

NoteProcessor is a good module candidate to support NIP33 queries (see #22) as event will have to be pruned out of the database, if they’re replaced.

@ariard ariard added this to the v0.0.2 milestone Jul 31, 2023
@ariard ariard force-pushed the 2023-07-impr-note-processor branch from f8dfa40 to bdde676 Compare July 31, 2023 02:38
@ariard
Copy link
Contributor Author

ariard commented Jul 31, 2023

Rebased at bdde676 after #24 merge.

@ariard ariard force-pushed the 2023-07-impr-note-processor branch from bdde676 to a9553fb Compare July 31, 2023 18:55
@ariard
Copy link
Contributor Author

ariard commented Jul 31, 2023

Updated at a9553fb with DB write on-disk.

@ariard ariard force-pushed the 2023-07-impr-note-processor branch 2 times, most recently from 037ebd0 to dd1d990 Compare August 1, 2023 18:49
@ariard
Copy link
Contributor Author

ariard commented Aug 1, 2023

Updated at dd1d990

The DB is functional and is saved on-disk. List of events received from the sample can be tested with list-db-events.

@ariard ariard force-pushed the 2023-07-impr-note-processor branch from dd1d990 to 732cead Compare August 2, 2023 17:50
@ariard ariard force-pushed the 2023-07-impr-note-processor branch from 732cead to 7b4cbf7 Compare August 2, 2023 17:52
@ariard
Copy link
Contributor Author

ariard commented Aug 2, 2023

Updated at 7b4cbf7 with CI fixed normally.

@ariard ariard merged commit a0c78d6 into civkit:main Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant