-
Notifications
You must be signed in to change notification settings - Fork 669
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
NOISSUE - Implement errors package in senml transformer, readers and writers #1108
Conversation
Signed-off-by: Ivan Milošević <iva@blokovi.com>
Codecov Report
@@ Coverage Diff @@
## master #1108 +/- ##
==========================================
- Coverage 76.96% 76.90% -0.06%
==========================================
Files 95 95
Lines 6823 6786 -37
==========================================
- Hits 5251 5219 -32
+ Misses 1231 1224 -7
- Partials 341 343 +2
Continue to review full report at Codecov.
|
Return wrapped error in postgres writer Signed-off-by: Ivan Milošević <iva@blokovi.com>
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.
LGTM
@@ -31,7 +31,7 @@ import ( | |||
) | |||
|
|||
const ( | |||
svcName = "postgres-writer" | |||
svcName = "postgres-reader" |
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.
Nice!
Signed-off-by: Ivan Milošević <iva@blokovi.com>
Signed-off-by: Ivan Milošević <iva@blokovi.com>
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.
LGTM
Signed-off-by: Ivan Milošević <iva@blokovi.com>
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.
LGTM
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.
LGTM
…writers (#1108) * Implement errors package in senml transformer, readers and writers Signed-off-by: Ivan Milošević <iva@blokovi.com> * Remove unused const Return wrapped error in postgres writer Signed-off-by: Ivan Milošević <iva@blokovi.com> * fix default db host in postgres writer Signed-off-by: Ivan Milošević <iva@blokovi.com> * fix capital letters in errors messages Signed-off-by: Ivan Milošević <iva@blokovi.com> * use svcName instead of postgres for Promethius initialization Signed-off-by: Ivan Milošević <iva@blokovi.com>
Signed-off-by: Ivan Milošević iva@blokovi.com
What does this do?
Use mainflux errors package in senml transformer, readers and writers
Which issue(s) does this PR fix/relate to?
Related to issues #538 and #870
List any changes that modify/break current functionality
Readers' API endpoints return error message in response body for errors (not just status response code in header)
Have you included tests for your changes?
Yes