-
Notifications
You must be signed in to change notification settings - Fork 7
docs(inkless): update readme with new sections and glossary #483
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
base: main
Are you sure you want to change the base?
Conversation
fcf81a5 to
11df5ae
Compare
3ade0f8 to
5fbc089
Compare
docs/inkless/GLOSSARY.md
Outdated
| ## B | ||
|
|
||
| **Batch** | ||
| In Inkless, a batch refers to a group of producer records that are written together as part of an object. Batches are the unit of coordination tracked by the Batch Coordinator. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For more clarity, I would add here that a batch comprises only messages from the same topic partition
docs/inkless/GLOSSARY.md
Outdated
| Configuration that enables clients (producers/consumers) and brokers to prefer operations within the same rack/AZ, minimizing cross-AZ data transfer costs. See [CLIENT-BROKER-AZ-ALIGNMENT.md](./CLIENT-BROKER-AZ-ALIGNMENT.md) for detailed implementation and configuration guide. | ||
|
|
||
| **Read Amplification** | ||
| When reading data for one partition requires fetching objects containing data from multiple partitions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's best to be more accurate and define what read amplification is in theory, and then how it applies to diskless like it's described here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea. Clarifying this on the last commit.
Co-authored-by: Giuseppe Lillo <giuseppe.lillo@aiven.io>
Co-authored-by: Giuseppe Lillo <giuseppe.lillo@aiven.io>
No description provided.