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

[Merged by Bors] - Give user feedback when consuming in interactive mode #1171

Closed

Conversation

nicholastmosher
Copy link
Contributor

Closes #700

When using the Consumer CLI in "interactive mode", the CLI will now print a little message about how it is going to consume.

image

The most important one here is in the case that no flags are given, e.g. fluvio consume one. This was confusing to the user in the past because it would appear to just hang as if it were loading, when it was actually consuming from the end of the partition and just waiting for records to arrive. Now there's a message to say that.

@sehz
Copy link
Contributor

sehz commented Jun 25, 2021

conflict

@nicholastmosher
Copy link
Contributor Author

@sehz should be ready now

Copy link
Contributor

@sehz sehz left a comment

Choose a reason for hiding this comment

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

nice improvement

@sehz
Copy link
Contributor

sehz commented Jun 27, 2021

bors r+

bors bot pushed a commit that referenced this pull request Jun 27, 2021
Closes #700 

When using the Consumer CLI in "interactive mode", the CLI will now print a little message about how it is going to consume.

<img width="669" alt="image" src="https://user-images.githubusercontent.com/4210949/122231966-e0f5a080-ce88-11eb-8073-d11d3ca12848.png">

The most important one here is in the case that no flags are given, e.g. `fluvio consume one`. This was confusing to the user in the past because it would appear to just hang as if it were loading, when it was actually consuming from the end of the partition and just waiting for records to arrive. Now there's a message to say that.
@bors
Copy link

bors bot commented Jun 27, 2021

Build failed:

@nicholastmosher
Copy link
Contributor Author

This seems to be spurious CI failure, going to try again

image

bors r+

bors bot pushed a commit that referenced this pull request Jun 28, 2021
Closes #700 

When using the Consumer CLI in "interactive mode", the CLI will now print a little message about how it is going to consume.

<img width="669" alt="image" src="https://user-images.githubusercontent.com/4210949/122231966-e0f5a080-ce88-11eb-8073-d11d3ca12848.png">

The most important one here is in the case that no flags are given, e.g. `fluvio consume one`. This was confusing to the user in the past because it would appear to just hang as if it were loading, when it was actually consuming from the end of the partition and just waiting for records to arrive. Now there's a message to say that.
@bors
Copy link

bors bot commented Jun 28, 2021

Build failed:

@nicholastmosher
Copy link
Contributor Author

bors r+

bors bot pushed a commit that referenced this pull request Jun 28, 2021
Closes #700 

When using the Consumer CLI in "interactive mode", the CLI will now print a little message about how it is going to consume.

<img width="669" alt="image" src="https://user-images.githubusercontent.com/4210949/122231966-e0f5a080-ce88-11eb-8073-d11d3ca12848.png">

The most important one here is in the case that no flags are given, e.g. `fluvio consume one`. This was confusing to the user in the past because it would appear to just hang as if it were loading, when it was actually consuming from the end of the partition and just waiting for records to arrive. Now there's a message to say that.
@bors
Copy link

bors bot commented Jun 28, 2021

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title Give user feedback when consuming in interactive mode [Merged by Bors] - Give user feedback when consuming in interactive mode Jun 28, 2021
@bors bors bot closed this Jun 28, 2021
@nicholastmosher nicholastmosher deleted the consumer-feedback branch August 27, 2021 14:52
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.

CLI: fluvio consume should give user feedback
2 participants