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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: check if we can remove the time.Sleep to wait the plugin load #4101

Closed
Pantani opened this issue Apr 24, 2024 · 0 comments 路 Fixed by #4179
Closed

fix: check if we can remove the time.Sleep to wait the plugin load #4101

Pantani opened this issue Apr 24, 2024 · 0 comments 路 Fixed by #4179
Assignees
Labels
component:apps Related to Ignite Apps. type:enhancement Improvement or update to existing feature(s).

Comments

@Pantani
Copy link
Collaborator

Pantani commented Apr 24, 2024

          It seems that adding the `KillClient` call might make the `time.Sleep` call redundant 馃, so it might be possible to remove the `time.Sleep`. Killing the client should try to stop it gracefully, so buffers should be flushed properly if the plugin client is running without issues.

Maybe you could add a TODO? If it can't be properly tested to be sure in fact the time.Sleep is redundant now.

Originally posted by @jeronimoalbi in #4091 (comment)

@salmad3 salmad3 added component:apps Related to Ignite Apps. type:enhancement Improvement or update to existing feature(s). labels May 29, 2024
@mergify mergify bot closed this as completed in #4179 Jun 5, 2024
mergify bot pushed a commit that referenced this issue Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:apps Related to Ignite Apps. type:enhancement Improvement or update to existing feature(s).
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants