Skip to content

Conversation

@freshtonic
Copy link
Contributor

@freshtonic freshtonic commented Oct 10, 2024

It's wordier than the doc for the other index types. I might have gone into too much detail - either that, or the other index types need more detail. Unsure.

@coderdan can you fact check my explanation of the purpose of the prefix (salt-like thingo) please?

Copy link
Contributor

@auxesis auxesis left a comment

Choose a reason for hiding this comment

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

This is great to see @freshtonic.

Have added some minor suggestions.

calvinbrewer and others added 4 commits October 10, 2024 08:24
Co-authored-by: Lindsay Holmwood <lindsay@holmwood.id.au>
Co-authored-by: Lindsay Holmwood <lindsay@holmwood.id.au>
Co-authored-by: Lindsay Holmwood <lindsay@holmwood.id.au>
###### ste_vec opts

An ste_vec index on a encrypted JSONB column enables the use of Postgres's `@>` and `<@` containment operators.
An ste_vec index on a encrypted JSONB column enables the use of PostgreSQL's `@>` and `<@` [containment operators](https://www.postgresql.org/docs/16/functions-json.html#FUNCTIONS-JSONB-OP-TABLE).
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nice

@freshtonic
Copy link
Contributor Author

Looks like @calvinbrewer has actioned your suggestions already, @auxesis .

Let me know when this is good to merge.

Copy link
Contributor

@auxesis auxesis left a comment

Choose a reason for hiding this comment

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

@freshtonic LGTM, :shipit:

@auxesis auxesis merged commit f09ae96 into main Oct 11, 2024
@auxesis auxesis deleted the ste_vec_docs branch October 11, 2024 01:49
tobyhede pushed a commit that referenced this pull request Oct 27, 2025
First pass at `ste_vec` docs for JSONB containment indexing
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.

4 participants