lamber-ken commented on a change in pull request #1730:
URL: https://github.com/apache/ozone/pull/1730#discussion_r553094609
##########
File path:
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/KeyValueContainer.java
##########
@@ -516,22 +516,36 @@ public void importContainerData(InputStream input,
@Override
public void exportContainerData(OutputStream destination,
ContainerPacker<KeyValueContainerData> packer) throws IOException {
- // Closed/ Quasi closed containers are considered for replication by
- // replication manager if they are under-replicated.
- ContainerProtos.ContainerDataProto.State state =
- getContainerData().getState();
- if (!(state == ContainerProtos.ContainerDataProto.State.CLOSED ||
- state == ContainerDataProto.State.QUASI_CLOSED)) {
- throw new IllegalStateException(
- "Only closed/quasi closed containers could be exported: " +
- "Where as ContainerId="
- + getContainerData().getContainerID() + " is in state " + state);
+ readLock();
Review comment:
> Agreed, the first read lock can be removed. I think we need to keep
the second one, though (for `packer.pack()`).
Agree +1
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
