Skip to content

feat: support pg13, pg14 as pgvector does#94

Merged
syvb merged 10 commits intotimescale:mainfrom
yihong0618:main
Oct 3, 2024
Merged

feat: support pg13, pg14 as pgvector does#94
syvb merged 10 commits intotimescale:mainfrom
yihong0618:main

Conversation

@yihong0618
Copy link
Contributor

This PR try to support pg12-pg14 as pgvector supports. some of the codes from zombodb

close: #93

This PR try to support pg12-pg14 as pgvector supports.
some of the codes from zombodb

close: timescale#93
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
Co-authored-by: zombodb <zombodb@gmail.com>
@yihong0618 yihong0618 requested a review from a team as a code owner June 13, 2024 11:52
@CLAassistant
Copy link

CLAassistant commented Jun 13, 2024

CLA assistant check
All committers have signed the CLA.

@cevian
Copy link
Collaborator

cevian commented Jun 14, 2024

@yihong0618 thanks. I'll review this.

@nickcanz
Copy link

nickcanz commented Jul 5, 2024

👋 Just 👍 this PR as having the same version support as pgvector would be very helpful for our use cases, we primarily use pg14 right now.

@syvb
Copy link
Contributor

syvb commented Sep 30, 2024

Could you remove PG12 support? I don't think it's worth it to add complexity for a PG version that will be obsolete in 2 months.

@yihong0618
Copy link
Contributor Author

Could you remove PG12 support? I don't think it's worth it to add complexity for a PG version that will be obsolete in 2 months.

OK wil do it these days

@yihong0618 yihong0618 changed the title feat: support pg12, pg13, pg14 as pgvector does feat: support pg13, pg14 as pgvector does Oct 1, 2024
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
@yihong0618
Copy link
Contributor Author

@syvb @cevian Done

Signed-off-by: yihong0618 <zouzou0208@gmail.com>
@yihong0618 yihong0618 requested a review from syvb October 3, 2024 01:25
syvb
syvb previously approved these changes Oct 3, 2024
Copy link
Contributor

@syvb syvb left a comment

Choose a reason for hiding this comment

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

Thanks!

Signed-off-by: Matvey Arye <cevian@gmail.com>
Copy link
Collaborator

@cevian cevian left a comment

Choose a reason for hiding this comment

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

Thanks so much for this change

@syvb syvb merged commit 659c100 into timescale:main Oct 3, 2024
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.

Supported Postgres versions

5 participants