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

create index on empty table #37

Closed

Conversation

davkhech
Copy link
Contributor

@davkhech davkhech commented Aug 9, 2023

when index is crated on empty table there is only header block
when new data is added, we add block map and data pages and populate them


CREATE TABLE small_world (
id varchar(3),
vector vector(3)
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you make this test with arrays and drop vector dependency?
And, use the small_world definition from test_helpers (just \ir test_helpers/small_world_arrays.sql)

test/sql/hnsw_insert_empty_table.sql Outdated Show resolved Hide resolved
@@ -1,4 +1,3 @@

#include "postgres.h"
Copy link
Contributor

Choose a reason for hiding this comment

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

Use <...> for postgres headers (#include "postgres.h")

@davkhech davkhech force-pushed the feature/create-index-on-empty-table branch from b844bb3 to 6e89143 Compare August 9, 2023 19:14
@davkhech davkhech marked this pull request as ready for review August 9, 2023 19:24
@Ngalstyan4
Copy link
Contributor

Merged these changes through your newer PR (#41). Thanks for your contribution!

@Ngalstyan4 Ngalstyan4 closed this Aug 11, 2023
var77 added a commit that referenced this pull request Oct 8, 2024
* - Create lock table in lantern schema
- Hash client trigger function and trigger names so they won't exceed the character limit

* Bump cli version
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.

2 participants