-
Notifications
You must be signed in to change notification settings - Fork 54
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
Error map for updateResults #914
Conversation
src/main/java/com/autotune/database/service/ExperimentDBService.java
Outdated
Show resolved
Hide resolved
The format that we have been using for all the APIs has been as below
@msvinaykumar I think we need to follow the same format for each individual record inside the bulk JSON as well. This means that we don't need two different ways to handle error conditions, one type for the entire bulk record and one for individual. Instead we need to have the same format for both. |
ok let me check |
Done |
...lyzer/performanceProfiles/PerformanceProfileInterface/ResourceOptimizationOpenshiftImpl.java
Outdated
Show resolved
Hide resolved
Signed-off-by: msvinaykumar <vinakuma@redhat.com>
Signed-off-by: msvinaykumar <vinakuma@redhat.com>
Signed-off-by: msvinaykumar <vinakuma@redhat.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
will get code with error message in updateResultsAPI