Skip to content

Conversation

@viragtripathi
Copy link
Contributor

Summary

This PR adds complete frontend UI configuration and CockroachDB Cloud support with SSL/TLS.

Changes

  • Frontend UI Configuration

    • Logo, parameter types, and input configurations
    • Loading and performance config for all index parameters
  • SSL/TLS Support for CockroachDB Cloud

    • Support verify-full, verify-ca, require, disable SSL modes
    • CLI parameters: --sslmode, --sslrootcert
    • SSL certificates passed to connection pool and optimize connections
  • Connection Handling Optimizations

    • Connection pooling for concurrent searches (enables true parallelism)
    • Settings in connection string to reduce per-connection overhead
    • Dedicated connection without statement_timeout for index creation
    • Handles background index creation with timeout detection and polling
  • CockroachDB-Specific Handling

    • Cancel running schema jobs before DROP TABLE (prevents blocking)
    • Transaction conflict retry logic for aborted transactions

Testing

Tested on both local multi-node cluster and CockroachDB Cloud with SSL verification.

- Add frontend Streamlit UI configuration
  - Logo, parameter types, and input configurations
  - Loading and performance config for all index parameters
- Add SSL/TLS support for CockroachDB Cloud clusters
  - Support verify-full, verify-ca, require, disable SSL modes
  - CLI parameters: --sslmode, --sslrootcert
  - SSL certificates passed to connection pool and optimize connections
- Optimize connection handling for multi-node clusters
  - Connection pooling for concurrent searches (enables true parallelism)
  - Settings in connection string to reduce per-connection overhead
  - Dedicated connection without statement_timeout for index creation
  - Handles background index creation with timeout detection and polling
- Handle CockroachDB-specific schema changes
  - Cancel running schema jobs before DROP TABLE (prevents blocking)
  - Transaction conflict retry logic for aborted transactions
- All linting checks passing
@viragtripathi
Copy link
Contributor Author

/assign @XuanYang-cn

@viragtripathi
Copy link
Contributor Author

@alwayslove2013 Please take a look and merge this enhancement/PR. Thank you!

@sre-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: alwayslove2013, viragtripathi
To complete the pull request process, please assign xuanyang-cn after the PR has been reviewed.
You can assign the PR to them by writing /assign @xuanyang-cn in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@alwayslove2013 alwayslove2013 merged commit 0da3d88 into zilliztech:main Dec 2, 2025
4 checks passed
Akhil-Pathivada pushed a commit to Akhil-Pathivada/VectorDBBench that referenced this pull request Dec 5, 2025
…illiztech#655)

- Add frontend Streamlit UI configuration
  - Logo, parameter types, and input configurations
  - Loading and performance config for all index parameters
- Add SSL/TLS support for CockroachDB Cloud clusters
  - Support verify-full, verify-ca, require, disable SSL modes
  - CLI parameters: --sslmode, --sslrootcert
  - SSL certificates passed to connection pool and optimize connections
- Optimize connection handling for multi-node clusters
  - Connection pooling for concurrent searches (enables true parallelism)
  - Settings in connection string to reduce per-connection overhead
  - Dedicated connection without statement_timeout for index creation
  - Handles background index creation with timeout detection and polling
- Handle CockroachDB-specific schema changes
  - Cancel running schema jobs before DROP TABLE (prevents blocking)
  - Transaction conflict retry logic for aborted transactions
- All linting checks passing
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.

3 participants