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

Option to view topic events based on offset range for a partition #1987

Closed
1 task
muralibasani opened this issue Nov 16, 2023 · 9 comments · Fixed by #2551
Closed
1 task

Option to view topic events based on offset range for a partition #1987

muralibasani opened this issue Nov 16, 2023 · 9 comments · Fixed by #2551
Labels
Backend Backend tasks

Comments

@muralibasani
Copy link
Contributor

What is currently missing?

Currently it is possible to view only last few selected offsets of partitions

How could this be improved?

It would be nice if user is given an option to provide an offsets range to view events

Is this a feature you would work on yourself?

  • I plan to open a pull request for this feature
@muralibasani muralibasani added the Backend Backend tasks label Nov 16, 2023
@khatibtamal
Copy link
Contributor

@muralibasani @aindriu-aiven This issue is tagged backend, does this mean you are only expecting a backend code update, and the UI will be done in separate issue? Or this issue needs to have the UI part as well?

@muralibasani
Copy link
Contributor Author

Hi @khatibtamal this also includes a UI change. But it could be better, if we have separate PRs for it.
Would you like to work on this ?

@khatibtamal
Copy link
Contributor

Hi @khatibtamal this also includes a UI change. But it could be better, if we have separate PRs for it. Would you like to work on this ?

Yes I would like to give this a go, I will initially open a PR for back end.
Thanks.

@khatibtamal
Copy link
Contributor

Hi @khatibtamal this also includes a UI change. But it could be better, if we have separate PRs for it. Would you like to work on this ?

Yes I would like to give this a go, I will initially open a PR for back end. Thanks.

Hey sorry was busy in the last few days, I will start working on this from tomorrow. Thanks.

@khatibtamal
Copy link
Contributor

@muralibasani I am trying to write Integration tests for my work, I am try to follow ClusterApiControllerIT.java but I cant run the test I get the following error. Is there a trick I am missing on how to run these integration tests? Thanks.

image

@muralibasani
Copy link
Contributor Author

@khatibtamal I will take a look and come back to you

@muralibasani
Copy link
Contributor Author

@khatibtamal You can try running it with mvn, from clusterapi foilder

mvn -Dtest=io.aiven.klaw.clusterapi.ClusterApiControllerIT surefire:test

@khatibtamal
Copy link
Contributor

@khatibtamal You can try running it with mvn, from clusterapi foilder

mvn -Dtest=io.aiven.klaw.clusterapi.ClusterApiControllerIT surefire:test

Hi thanks it worked.

@khatibtamal
Copy link
Contributor

@muralibasani PR is open for the back end. Thanks.
cc @aindriu-aiven

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend Backend tasks
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants