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
Fixes #2942 - Fix bugs reported by Sonar issue on release 0.9 #2943
Conversation
@@ -38,7 +37,7 @@ | |||
public class PermissionsResource { | |||
private final Authorizer authorizer; | |||
|
|||
public PermissionsResource(CollectionDAO dao, @NonNull Authorizer authorizer) { | |||
public PermissionsResource(@NonNull Authorizer authorizer) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will result in runtime error from CollectionRegistry to register this resource, since it looks for constructor with two args in the order CollectionDAO and Authorizer
This does not fail during compile time..
I should have left a comment here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay reverted that change with suppressing unused warning annotation...
dd0c793
to
b9a4cba
Compare
[open-metadata-ingestion] Kudos, SonarCloud Quality Gate passed! |
[catalog-rest-service] Kudos, SonarCloud Quality Gate passed! 9 Bugs |
SonarCloud issues are unrelated to this patch. |
Describe your changes :
See #2942 for more details
Type of change :
Checklist: