Skip to content

docs: clarify which properties are vectorized (source_properties + blob)#456

Open
g-despot wants to merge 1 commit into
mainfrom
docs/clarify-source-properties-vectorization
Open

docs: clarify which properties are vectorized (source_properties + blob)#456
g-despot wants to merge 1 commit into
mainfrom
docs/clarify-source-properties-vectorization

Conversation

@g-despot

Copy link
Copy Markdown
Contributor

The Configure semantic indexing section stated that only text/text[] properties are vectorized. Two corrections:

  • A blob is a base64-encoded string, so blob properties are vectorized by default too — added a caution on how to exclude them.
  • When source_properties is set, listed non-text properties (number, int, boolean, date, object, and array variants) are also vectorized. This wasn't documented; added a "Specify which properties to vectorize" subsection.

Also notes that uuid, geo-coordinate, and phone-number properties are never vectorized.

The 'Configure semantic indexing' section said only text/text[] are vectorized.
Two corrections: blob (a base64 string) is also vectorized by default, and when
source_properties is set, listed non-text properties (number/int/bool/date/object
and array variants) are vectorized too. Adds a 'Specify which properties to
vectorize' subsection and a blob caution.

@orca-security-eu orca-security-eu Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

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