Github user zuyu commented on a diff in the pull request:
https://github.com/apache/incubator-quickstep/pull/301#discussion_r141768111
--- Diff: transaction/LockManager.cpp ---
@@ -79,22 +79,23 @@ void LockManager::run() {
const LockRequest request = incoming_requests_.popOne();
if (request.getRequestType() == RequestType::kReleaseLocks) {
CHECK(releaseAllLocks(request.getTransactionId()))
- << "Unexpected condition occured.";
-
+ << "Unexpected condition occured.";
} else if (acquireLock(request.getTransactionId(),
- request.getResourceId(),
- request.getAccessMode())) {
+ request.getResourceId(),
+ request.getAccessMode())) {
--- End diff --
Please align with the first argument.
---
