-
Notifications
You must be signed in to change notification settings - Fork 517
# feat: Improve fluvio topic describe with additional information #3968 #4508
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
gmkumar2005
commented
Apr 14, 2025
- Displays both topic metadata and its partitions in a single command
- Includes a --system flag to optionally show system partitions
…inyon#3968 - Displays both topic metadata and its partitions in a single command - Includes a --system flag to optionally show system partitions
How to filter topics in admin api ? The following call get list of topics how ever it is not able to get filtered list
Code with name filter
The value list_filters does not have any effect, the admin returns all the topics.
Please advice if there is a better approach ? |
…inyon#3968 - Displays both topic metadata and its partitions in a single command - Includes a --system flag to optionally show system partitions
…inyon#3968 - Displays both topic metadata and its partitions in a single command - Includes a --system flag to optionally show system partitions
Sample output
Sample output of system topic
|
Looks like SC partition doesn't properly filter on topic name: https://github.com/infinyon/fluvio/blob/master/crates/fluvio-sc/src/services/public_api/partition/mod.rs. It'e straightforward to add them like: https://github.com/infinyon/fluvio/blob/master/crates/fluvio-sc/src/services/public_api/topic/fetch.rs |
@sehz Fixing the code in public_api for partitions requires some more analysis, I ran into type related issues. |
Sounds good |
For CLI, need to add CLI E2E test, You can find them here: https://github.com/infinyon/fluvio/tree/master/tests/cli/fluvio_smoke_tests |
- added test case for `filter_partition_by_topic`
- added test case for `filter_partition_by_topic`
Added smoke test |
Followup PR for updating the docs |