Do not clear d_api context maps during the close

mykola-semko · facebook-github-bot · commit af9b8db6be4d · 2021-10-13T09:51:26.000-07:00
Summary: ^

Reviewed By: ndmitchell

fbshipit-source-id: 5e3b72f5aeb8657fe30b88d2c5134d30b6e36b54
diff --git a/src/com/facebook/buck/downwardapi/processexecutor/context/DownwardApiExecutionContext.java b/src/com/facebook/buck/downwardapi/processexecutor/context/DownwardApiExecutionContext.java
@@ -118,9 +118,6 @@ public void registerActionId(ActionId actionId) {
   @Override
   public synchronized void close() {
     verifyAllEventsProcessed();
-    chromeTraceStartedEvents.clear();
-    stepStartedEvents.clear();
-    actionToThreadIdMap.clear();
   }
 
   private void verifyAllEventsProcessed() {

Original file line number	Diff line number	Diff line change
`@@ -118,9 +118,6 @@ public void registerActionId(ActionId actionId) {`
`118`	`118`	`@Override`
`119`	`119`	`public synchronized void close() {`
`120`	`120`	`verifyAllEventsProcessed();`
`121`		`- chromeTraceStartedEvents.clear();`
`122`		`- stepStartedEvents.clear();`
`123`		`- actionToThreadIdMap.clear();`
`124`	`121`	`}`
`125`	`122`
`126`	`123`	`private void verifyAllEventsProcessed() {`