javeme commented on code in PR #1896:
URL:
https://github.com/apache/incubator-hugegraph/pull/1896#discussion_r914608827
##########
hugegraph-hbase/src/main/java/com/baidu/hugegraph/backend/store/hbase/HbaseSessions.java:
##########
@@ -554,7 +552,7 @@ public Integer commit() {
} catch (Throwable e) {
// TODO: Mark and delete committed records
throw new BackendException("Failed to commit, " +
- "there may be inconsistent states for HBase", e);
+ "there may be inconsistent
states for HBase", e);
Review Comment:
exceed 100 chars?
##########
hugegraph-hbase/src/main/java/com/baidu/hugegraph/backend/store/hbase/HbaseSessions.java:
##########
@@ -809,7 +807,7 @@ private void dump(String table, Scan scan) throws
IOException {
byte[] key = CellUtil.cloneQualifier(cell);
byte[] val = CellUtil.cloneValue(cell);
LOG.info(" {}={}", StringEncoding.format(key),
- StringEncoding.format(val));
+ StringEncoding.format(val));
Review Comment:
also wrap line before `StringEncoding.format(key)` for logic symmetry
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
