[tez] branch master updated: TEZ-4251: Acquiring locks for getInputVertices and getOutputVertices is not consistent (Krisztian Kasa reviewed by Rajesh Balamohan)

[rbalamohan]

This is an automated email from the ASF dual-hosted git repository.

rbalamohan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tez.git


The following commit(s) were added to refs/heads/master by this push:
     new 938d6a1  TEZ-4251: Acquiring locks for getInputVertices and 
getOutputVertices is not consistent (Krisztian Kasa reviewed by Rajesh 
Balamohan)
938d6a1 is described below

commit 938d6a105c398531774a8c341dc851b7ceee4eeb
Author: Rajesh Balamohan <rbalamo...@apache.org>
AuthorDate: Tue Nov 24 13:10:15 2020 +0530

    TEZ-4251: Acquiring locks for getInputVertices and getOutputVertices is not 
consistent (Krisztian Kasa reviewed by Rajesh Balamohan)
---
 .../src/main/java/org/apache/tez/dag/app/dag/impl/VertexImpl.java  | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git 
a/tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/VertexImpl.java 
b/tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/VertexImpl.java
index 711d028..f86dc98 100644
--- a/tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/VertexImpl.java
+++ b/tez-dag/src/main/java/org/apache/tez/dag/app/dag/impl/VertexImpl.java
@@ -4460,12 +4460,7 @@ public class VertexImpl implements 
org.apache.tez.dag.app.dag.Vertex, EventHandl
 
   @Override
   public Map<Vertex, Edge> getInputVertices() {
-    readLock.lock();
-    try {
-      return Collections.unmodifiableMap(this.sourceVertices);
-    } finally {
-      readLock.unlock();
-    }
+    return Collections.unmodifiableMap(this.sourceVertices);
   }
 
   @Override