Michael Blow has submitted this change and it was merged. Change subject: [NO ISSUE][OTR] Logging Cleanup ......................................................................
[NO ISSUE][OTR] Logging Cleanup - user model changes: no - storage format changes: no - interface changes: no Details: - No pretty printing for replica status. - Log ignored exception in debug level. Change-Id: I7e4a37e9621ef5bdb27105b17e883800212e594b Reviewed-on: https://asterix-gerrit.ics.uci.edu/2276 Sonar-Qube: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> Integration-Tests: Jenkins <[email protected]> Reviewed-by: Ian Maxon <[email protected]> --- M asterixdb/asterix-replication/src/main/java/org/apache/asterix/replication/api/PartitionReplica.java M hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/dataflow/IndexDropOperatorNodePushable.java 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Anon. E. Moose #1000171: Jenkins: Verified; No violations found; Verified Ian Maxon: Looks good to me, approved diff --git a/asterixdb/asterix-replication/src/main/java/org/apache/asterix/replication/api/PartitionReplica.java b/asterixdb/asterix-replication/src/main/java/org/apache/asterix/replication/api/PartitionReplica.java index f752745..d3ddc43 100644 --- a/asterixdb/asterix-replication/src/main/java/org/apache/asterix/replication/api/PartitionReplica.java +++ b/asterixdb/asterix-replication/src/main/java/org/apache/asterix/replication/api/PartitionReplica.java @@ -128,7 +128,7 @@ private JsonNode asJson() { ObjectNode json = OBJECT_MAPPER.createObjectNode(); json.put("id", id.toString()); - json.put("state", status.name()); + json.put("status", status.name()); return json; } @@ -152,7 +152,7 @@ @Override public String toString() { try { - return JSONUtil.convertNode(asJson()); + return OBJECT_MAPPER.writeValueAsString(asJson()); } catch (JsonProcessingException e) { throw new ReplicationException(e); } diff --git a/hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/dataflow/IndexDropOperatorNodePushable.java b/hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/dataflow/IndexDropOperatorNodePushable.java index c59c1a8..f75144a 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/dataflow/IndexDropOperatorNodePushable.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/dataflow/IndexDropOperatorNodePushable.java @@ -83,7 +83,7 @@ return; } catch (HyracksDataException e) { if (isIgnorable(e)) { - LOGGER.info("Ignoring exception on drop", e); + LOGGER.debug("Ignoring exception on drop", e); return; } if (canRetry(e)) { -- To view, visit https://asterix-gerrit.ics.uci.edu/2276 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7e4a37e9621ef5bdb27105b17e883800212e594b Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]>
