Skip to content

Narek/handle deletes#317

Merged
Ngalstyan4 merged 5 commits intomainfrom
narek/handle-deletes
May 4, 2024
Merged

Narek/handle deletes#317
Ngalstyan4 merged 5 commits intomainfrom
narek/handle-deletes

Conversation

@Ngalstyan4
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

7 similar comments
@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@codecov
Copy link
Copy Markdown

codecov bot commented May 3, 2024

Codecov Report

Attention: Patch coverage is 89.47368% with 4 lines in your changes are missing coverage. Please review.

Files Patch % Lines
src/hnsw/delete.c 89.65% 0 Missing and 3 partials ⚠️
src/hnsw/scan.c 83.33% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

11 similar comments
@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 3, 2024

Benchmarks

metric old new pct change
recall (after create) 0.948 0.955 +0.74%
recall (after insert) 0.000 0.000 -
select tps 19529.661 22371.365 +14.55%
select bulk(100) tps 33.739 34.950 +3.59%
select latency (ms) 0.852 ± 2.074𝜎 0.820 ± 1.630𝜎 -3.76%
select bulk(100) latency (ms) 842.798 ± 160.926𝜎 835.402 ± 166.157𝜎 -0.88%
create latency (ms) 366305.604 369841.874 +0.97%
insert tps 483.653 493.207 +1.98%
insert bulk(100) tps 4.962 5.174 +4.27%
insert latency (ms) 65.384 ± 18.408𝜎 63.977 ± 17.833𝜎 -2.15%
insert bulk(100) latency (ms) 6310.746 ± 201.929𝜎 6006.067 ± 213.679𝜎 -4.83%
disk usage (bytes) 8196194304.000 8196194304.000 -

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

5 similar comments
@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@Ngalstyan4 Ngalstyan4 force-pushed the narek/handle-deletes branch from 99f0fe8 to 54bb46e Compare May 4, 2024 01:19
@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

9 similar comments
@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@Ngalstyan4 Ngalstyan4 force-pushed the narek/handle-deletes branch from 54bb46e to dfe48c2 Compare May 4, 2024 01:31
Ngalstyan4 added 4 commits May 4, 2024 01:47
This is necessary to make sure that we do not return irrelevant
and wrong results when eventually postgres reuses tids
- Updates for mentioned versions are archived, so the checks are
unnecessary
- There was a todo to revert current version mismatch warning into an
  error. As we move to support live upgrades, it is crucial to
maintain cross-version binary compatibility as that greatly simplifies
the update process. So, we will not be moving away from the warning
approach
add requirements txt
@Ngalstyan4 Ngalstyan4 force-pushed the narek/handle-deletes branch from dfe48c2 to 01de4d5 Compare May 4, 2024 01:47
@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

9 similar comments
@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 4, 2024

The update tests on Linux failed. Please check the logs for more details.

@Ngalstyan4 Ngalstyan4 merged commit cd7f187 into main May 4, 2024
@Ngalstyan4 Ngalstyan4 deleted the narek/handle-deletes branch May 4, 2024 01:56
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