Skip to content

feat: add Columnar Engine, Auto-Columnarization, and Vector Assist op… - #629

Open
madamsetty-pavan wants to merge 12 commits into
googleapis:mainfrom
madamsetty-pavan:feat/columnar-engine
Open

feat: add Columnar Engine, Auto-Columnarization, and Vector Assist op…#629
madamsetty-pavan wants to merge 12 commits into
googleapis:mainfrom
madamsetty-pavan:feat/columnar-engine

Conversation

@madamsetty-pavan

@madamsetty-pavan madamsetty-pavan commented Aug 12, 2026

Copy link
Copy Markdown

feat: add Columnar Engine, Auto-Columnarization, and Vector Assist optimizations

  • Add enable_columnar_engine and aenable_columnar_engine
  • Add enable_auto_columnarization and aenable_auto_columnarization
  • Add Vector Assist spec definition, application, and recommendations
  • Add initialize_auto_vector_embeddings for automated embedding generation
  • Add comprehensive sync and async unit tests with detailed comments

…timizations

- Add enable_columnar_engine and aenable_columnar_engine
- Add enable_auto_columnarization and aenable_auto_columnarization
- Add Vector Assist spec definition, application, and recommendations
- Add initialize_auto_vector_embeddings for automated embedding generation
- Add comprehensive sync and async unit tests with detailed comments
@product-auto-label product-auto-label Bot added the api: alloydb Issues related to the googleapis/langchain-google-alloydb-pg-python API. label Aug 12, 2026
@madamsetty-pavan
madamsetty-pavan marked this pull request as ready for review August 26, 2026 23:44
@madamsetty-pavan
madamsetty-pavan requested review from a team as code owners August 26, 2026 23:44
schema_name (str): The schema name of the table. Default: "public".
"""
await self._run_as_async(
self._adrop_table(table_name=table_name, schema_name=schema_name)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Is _adrop_table implemented anywhere?

params = "WITH " + index.index_options()
if name is None:
if index.name is None:
index.name = self.table_name + DEFAULT_INDEX_NAME_SUFFIX

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Not sure if index.name is ever used elsewhere, but just confirming that index.name mutation is intentional.

@zhou0ricky zhou0ricky 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.

Added a few comments, otherwise should be good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: alloydb Issues related to the googleapis/langchain-google-alloydb-pg-python API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants