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

ServiceTask state set to success even if the task actually fails #87

Closed
amischler opened this Issue Nov 27, 2013 · 3 comments

Comments

Projects
None yet
2 participants
@amischler

There is a try/catch in the call method that log the failure of the task, but the exception is not propagated and the task ended with a success state.

@ghost ghost assigned sbordes Nov 27, 2013

@sbordes

This comment has been minimized.

Show comment
Hide comment
@sbordes

sbordes Nov 27, 2013

Member

OK I will fix this issue into the 0.7.6, it's linked with the #86

I will rollback all jars deployed because I'm waiting a bug fixing from Bintray to be able to deploy to Maven Central the first JRebirth artifacts

Member

sbordes commented Nov 27, 2013

OK I will fix this issue into the 0.7.6, it's linked with the #86

I will rollback all jars deployed because I'm waiting a bug fixing from Bintray to be able to deploy to Maven Central the first JRebirth artifacts

@amischler

This comment has been minimized.

Show comment
Hide comment
@amischler

amischler Nov 27, 2013

Ok, thank you. To my mind, it would be better to release a new 0.7.7 for these bugfixes than updating the 0.7.6 released jars. Actually, maven et maven repositories and proxies never update a released jar, so it has be done manually everywhere and this can lead to pretty confusing situations.

Ok, thank you. To my mind, it would be better to release a new 0.7.7 for these bugfixes than updating the 0.7.6 released jars. Actually, maven et maven repositories and proxies never update a released jar, so it has be done manually everywhere and this can lead to pretty confusing situations.

@sbordes

This comment has been minimized.

Show comment
Hide comment
@sbordes

sbordes Nov 27, 2013

Member

ok these 2 features will be added into 0.7.7, that will be the first version released on central (I hope)

Member

sbordes commented Nov 27, 2013

ok these 2 features will be added into 0.7.7, that will be the first version released on central (I hope)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment