Skip to content

Conversation

@TropicolX
Copy link
Contributor

@TropicolX TropicolX commented Nov 22, 2025

Note

Expands vt6 stage docs with detailed guidance for DescribeTopicPartitions (unknown topic), including request parsing, response header v1, response body structure, example bytes, and updated tester steps/validations.

  • Docs (vt6 stage):
    • Add detailed overview of DescribeTopicPartitions API and focus on handling unknown topics.
    • Introduce request parsing to extract topic_name from topics (compact array).
    • Switch to and document response header v1 (correlation_id, TAG_BUFFER).
    • Specify response body schema and required values for unknown topics:
      • error_code = 3, echo topic_name, zero topic_id, is_internal=false, empty partitions, next_cursor=-1.
    • Provide a byte-level example response for topic "foo".
    • Update test instructions and validations (netcat command, message_size correctness, header v1 usage, field expectations).

Written by Cursor Bugbot for commit 1995bdb. This will update automatically on new commits. Configure here.

@github-actions
Copy link

github-actions bot commented Nov 22, 2025

LLM Doc Lint

15 tests   12 ✅  0s ⏱️
 1 suites   0 💤
 1 files     3 ❌

For more details on these failures, see this check.

Results for commit ba42e7d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Nov 22, 2025

LLM Doc Lint

15 tests   14 ✅  0s ⏱️
 1 suites   0 💤
 1 files     1 ❌

For more details on these failures, see this check.

Results for commit cb41241.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Nov 22, 2025

LLM Doc Lint

15 tests   15 ✅  0s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit 9ac3209.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Nov 23, 2025

LLM Doc Lint

15 tests   15 ✅  0s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit 1995bdb.

♻️ This comment has been updated with latest results.

@TropicolX TropicolX merged commit 7e8d419 into main Nov 23, 2025
20 of 21 checks passed
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