Skip to content

chore: cleanup after docs review#313

Open
fresh-borzoni wants to merge 3 commits intoapache:mainfrom
fresh-borzoni:fix-inconsistencies-2
Open

chore: cleanup after docs review#313
fresh-borzoni wants to merge 3 commits intoapache:mainfrom
fresh-borzoni:fix-inconsistencies-2

Conversation

@fresh-borzoni
Copy link
Contributor

Summary

  • Rename OffsetQuery (C++) and OffsetType (Python) to OffsetSpec with factory methods, matching the Java client API
  • Add list_partition_infos with optional partial partition spec filter to Python and C++ clients
  • Standardize Python config dict keys to snake_case (bootstrap_servers, writer_batch_size, etc.) matching C++/Rust field names
  • Update docs, type stubs, and examples to reflect all changes

@fresh-borzoni
Copy link
Contributor Author

@leekeiabstraction PTAL 🙏

@fresh-borzoni fresh-borzoni mentioned this pull request Feb 13, 2026
Copy link
Contributor

@leekeiabstraction leekeiabstraction left a comment

Choose a reason for hiding this comment

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

TY for the PR! Left some comments PTAL

@fresh-borzoni
Copy link
Contributor Author

@leekeiabstraction Ty for the review. Addressed comment about py configs

Copy link
Contributor

@leekeiabstraction leekeiabstraction left a comment

Choose a reason for hiding this comment

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

Left comment, thank you for the update!

@fresh-borzoni
Copy link
Contributor Author

@luoyuxia Sorry to bother, but we might need your help here.

@fresh-borzoni
Copy link
Contributor Author

@leekeiabstraction @luoyuxia Ty for the review
rebased, updated PTAL 🙏

Copy link
Contributor

@leekeiabstraction leekeiabstraction left a comment

Choose a reason for hiding this comment

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

LGTM, TY for the PR!

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