Skip to content
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

Setting offsets by timestamp fails without the topic name option #192

Open
ognjen-j opened this issue Jul 22, 2021 · 0 comments
Open

Setting offsets by timestamp fails without the topic name option #192

ognjen-j opened this issue Jul 22, 2021 · 0 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@ognjen-j
Copy link
Contributor

When using set offsets for a certain timestamp, the command fails if no topic name is provided (-t option). Since everything works correctly when using an absolute value as an offset, the error is presumably caused by the part of the function that fetches the closest offsets for the given timestamp.
If the topic name is omitted, esque should assume all topics for which the consumer has offsets (the same way it already does with absolute offset values).

@ognjen-j ognjen-j added bug Something isn't working good first issue Good for newcomers labels Jul 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant