Skip to content

[1.13] QuestDB Docs#211

Open
akashverma0786 wants to merge 1 commit into
open-metadata:mainfrom
akashverma0786:Questdb-docs
Open

[1.13] QuestDB Docs#211
akashverma0786 wants to merge 1 commit into
open-metadata:mainfrom
akashverma0786:Questdb-docs

Conversation

@akashverma0786
Copy link
Copy Markdown
Contributor

No description provided.

@ayush-shah
Copy link
Copy Markdown
Member

Code review note from May 5, 2026: blocking before merge.

Findings:

  • The PR adds QuestDB docs, but the current OpenMetadata main checkout does not contain a QuestDB connector/schema (no questdbConnection or ingestion source found). Please confirm the matching connector code is merged into the target release before publishing the docs.
  • The PR adds v1.13.x-SNAPSHOT QuestDB pages and docs.json navigation, but does not add QuestDB to the v1.13 connector list snippet. The v1.12 connector list is updated, so the latest connector grid would omit QuestDB.
  • The PR currently has no status checks showing in GitHub; Mintlify validation should run before merge.

Recommendation: hold merge until the connector code/release dependency is confirmed, the v1.13 connector list is updated, and validation runs.

@ayush-shah ayush-shah changed the title QuestDB Docs [1.13] QuestDB Docs May 11, 2026
Copy link
Copy Markdown
Member

@ayush-shah ayush-shah left a comment

Choose a reason for hiding this comment

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

Revalidated current head 851b8beb.

Validation:

  • rtk mint broken-links: passed.
  • git diff --check origin/main...refs/remotes/pr/211: clean.
  • Cross-check: local Collate/OpenMetadata product checkouts still do not contain a QuestDB connector/schema, and snippets/v1.13.x-SNAPSHOT/connectors/database/connectors-list.mdx is still not updated even though v1.13 docs/navigation pages were added.

The May 5 blocking note still applies: please confirm the connector code/release dependency and add QuestDB to the v1.13 connector list before merge.

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.

2 participants