yujunjun/datahub
2
0
Fork 0
mirror of https://github.com/datahub-project/datahub.git synced 2025-12-30 03:18:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add more logging for owner and compliance API (#708)

Browse Source
This commit is contained in:
Yi (Alan) Wang 2017-08-26 11:48:24 -07:00 committed by GitHub
parent 1c0eeea181
commit 4f2c7d6c2e
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
wherehows-frontend/app/controllers/api/v1/Dataset.java
View File

@ -201,6 +201,7 @@ public class Dataset extends Controller {
result.set("owners", Json.toJson(DATASETS_DAO.getDatasetOwnersByID(JDBC_TEMPLATE, NAMED_JDBC_TEMPLATE, id)));
result.put("status", "ok");
} catch (Exception e) {
Logger.warn("Failed to get owners: " + e.toString());
result.put("status", "failed");
result.put("message", "Error: " + e.getMessage());
}
@ -281,6 +282,7 @@ public class Dataset extends Controller {
DATASETS_DAO.updateDatasetOwners(JDBC_TEMPLATE, username, id, owners);
result.put("status", "success");
} catch (Exception e) {
Logger.error("Owner updating error: ", e);
result.put("status", "failed");
result.put("error", "true");
result.put("msg", "Could not update dataset owners: " + e.getMessage());
@ -755,6 +757,7 @@ public class Dataset extends Controller {
try {
record = DATASETS_DAO.getDatasetComplianceInfoByDatasetId(JDBC_TEMPLATE, datasetId);
} catch (Exception e) {
Logger.warn("Failed to get compliance: " + e.toString());
JsonNode result = Json.newObject()
.put("status", "failed")
.put("error", "true")