YARN-10296. Make ContainerPBImpl#getId/setId synchronized. Contributed by Benjamin Teke
This commit is contained in:
parent
034d458511
commit
8bef26a607
|
@ -63,7 +63,7 @@ public class ContainerPBImpl extends Container {
|
|||
viaProto = true;
|
||||
}
|
||||
|
||||
public ContainerProto getProto() {
|
||||
synchronized public ContainerProto getProto() {
|
||||
|
||||
mergeLocalToProto();
|
||||
proto = viaProto ? proto : builder.build();
|
||||
|
@ -132,7 +132,7 @@ public class ContainerPBImpl extends Container {
|
|||
}
|
||||
|
||||
@Override
|
||||
public ContainerId getId() {
|
||||
synchronized public ContainerId getId() {
|
||||
ContainerProtoOrBuilder p = viaProto ? proto : builder;
|
||||
if (this.containerId != null) {
|
||||
return this.containerId;
|
||||
|
@ -166,7 +166,7 @@ public class ContainerPBImpl extends Container {
|
|||
}
|
||||
|
||||
@Override
|
||||
public void setId(ContainerId id) {
|
||||
synchronized public void setId(ContainerId id) {
|
||||
maybeInitBuilder();
|
||||
if (id == null)
|
||||
builder.clearId();
|
||||
|
|
Loading…
Reference in New Issue