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

docs: first API draft #1

Merged
merged 7 commits into from
Dec 1, 2023
Merged

docs: first API draft #1

merged 7 commits into from
Dec 1, 2023

Conversation

ttypic
Copy link
Collaborator

@ttypic ttypic commented Nov 14, 2023

No description provided.

@ttypic ttypic marked this pull request as ready for review November 15, 2023 17:48
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@AndyTWF
Copy link
Collaborator

AndyTWF commented Nov 17, 2023

A bit of a general question - how do we see the customer application interacting with the library? Lets say I wanted to "scroll up" my chat and get some older messages - would I just do a query at appropriate points to populate the messages?

How would huge volumes of messages be handled - lets say a user decided to scroll all the way up a chat (containing say millions of messages), would the client library try to hold every message in memory? Or would we make it some sort of sparse cache, that may not contain all messages - but is guaranteed to contain the latest version of a message if it does have it?

That sorta leads me on to a thought - do we make the client library implement a "window" of messages? Like, the customer tells the library which window of messages the user is currently in, and the library only generates events for messages in that window? Thus it keeps the number of messages the library has in memory to a fixed size?

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@ttypic ttypic requested a review from AndyTWF November 24, 2023 11:59
@ttypic ttypic force-pushed the first-draft branch 2 times, most recently from 53b1e4e to 7c9dbc8 Compare December 1, 2023 12:27
@ttypic ttypic merged commit 3faca8a into main Dec 1, 2023
6 checks passed
@ttypic ttypic deleted the first-draft branch December 4, 2023 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants