imbajin commented on code in PR #2200:
URL:
https://github.com/apache/incubator-hugegraph/pull/2200#discussion_r1170806056
##########
hugegraph-rocksdb/src/main/java/org/apache/hugegraph/backend/store/rocksdb/RocksDBStdSessions.java:
##########
@@ -299,24 +295,20 @@ public String buildSnapshotPath(String snapshotPrefix) {
// Like: rocksdb-data/*
Path pureDataPath = parentParentPath.relativize(originDataPath);
// Like: parent_path/snapshot_rocksdb-data/*
- Path snapshotPath = parentParentPath.resolve(snapshotPrefix + "_" +
- pureDataPath);
+ Path snapshotPath = parentParentPath.resolve(snapshotPrefix + "_" +
pureDataPath);
E.checkArgument(snapshotPath.toFile().exists(),
- "The snapshot path '%s' doesn't exist",
- snapshotPath);
+ "The snapshot path '%s' doesn't exist", snapshotPath);
Review Comment:
> it's more clear to keep 3 lines
seems one phrase for one line to read is better? like
```java
// ① not split
throw new BackendException("Table '%s' is not opened", cfName);
// ② split, not necessary to break one line unless it's too long?
throw new BackendException("Table '%s' is not opened",
cfName);
```
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
