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

[blog] Version 1.13 Release #1396

Merged
merged 23 commits into from
Jan 23, 2025
Merged

[blog] Version 1.13 Release #1396

merged 23 commits into from
Jan 23, 2025

Conversation

davidmyriel
Copy link
Contributor

@davidmyriel davidmyriel commented Jan 16, 2025

Copy link

netlify bot commented Jan 16, 2025

Deploy Preview for condescending-goldwasser-91acf0 ready!

Name Link
🔨 Latest commit 423cda7
🔍 Latest deploy log https://app.netlify.com/sites/condescending-goldwasser-91acf0/deploys/67923cd1fec78c0008652ca9
😎 Deploy Preview https://deploy-preview-1396--condescending-goldwasser-91acf0.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

Choose a reason for hiding this comment

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

Nice visualizer 👍

However, if we are looking for correctness, this image is a bit misleading.

Correction:

  • The Mask layer includes the regions. Regions are there to "summarize" the bits in the bitmask, and the bitmask is there to summarize the blocks. The purpose of this layer is to know where to place updates.

  • The Tracker layer directly links an ID to a position and length of the data in the Data layer. The purpose of this layer is reading data.

Copy link
Member

Choose a reason for hiding this comment

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

What exactly is wrong?

Copy link
Member

Choose a reason for hiding this comment

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

Oh, the tracking layer is not what I though.
How to call this "additional structure which tracks gaps in regions of the bitmask"?

I don't think we need to mention tacker separately then, cause it is trivial

Copy link
Contributor

@coszio coszio Jan 20, 2025

Choose a reason for hiding this comment

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

I see, so the misleading part is only naming-wise for me. Which I associated Tracker with the lookup part.

How to call this "additional structure which tracks gaps in regions of the bitmask"?

Regions Layer (?)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It fits the attached diagram...

qdrant-landing/content/blog/qdrant-1.13.x.md Outdated Show resolved Hide resolved
qdrant-landing/content/blog/qdrant-1.13.x.md Outdated Show resolved Hide resolved
qdrant-landing/content/blog/qdrant-1.13.x.md Outdated Show resolved Hide resolved
qdrant-landing/content/blog/qdrant-1.13.x.md Outdated Show resolved Hide resolved
qdrant-landing/content/blog/qdrant-1.13.x.md Outdated Show resolved Hide resolved
qdrant-landing/content/blog/qdrant-1.13.x.md Outdated Show resolved Hide resolved
qdrant-landing/content/blog/qdrant-1.13.x.md Outdated Show resolved Hide resolved
qdrant-landing/content/blog/qdrant-1.13.x.md Outdated Show resolved Hide resolved
qdrant-landing/content/blog/qdrant-1.13.x.md Outdated Show resolved Hide resolved
qdrant-landing/content/blog/qdrant-1.13.x.md Outdated Show resolved Hide resolved
@davidmyriel davidmyriel merged commit de469a9 into master Jan 23, 2025
5 checks passed
@davidmyriel davidmyriel deleted the qdrant-1.13 branch January 23, 2025 13:01
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.

5 participants