Ignore NotInTransactionException while obtaining stats

If query gets cancelled during planning transaction is removed from
transaction manager and we start getting errors while trying to get
table stats. Ignore such failures to avoid log pollution.

core/trino-main/src/main/java/io/trino/cost/CachingStatsProvider.java

@@ -14,6 +14,7 @@
 package io.trino.cost;
 
 import io.airlift.log.Logger;
+import io.trino.NotInTransactionException;
 import io.trino.Session;
 import io.trino.sql.planner.iterative.GroupReference;
 import io.trino.sql.planner.iterative.Lookup;
@@ -89,14 +90,29 @@ public PlanNodeStatsEstimate getStats(PlanNode node)
             return stats;
         }
         catch (RuntimeException e) {
+            if (shouldIgnoreStatsCalculatorFailure(e)) {
+                // some failures are okay to ignore
+                return PlanNodeStatsEstimate.unknown();
+            }
+
             if (isIgnoreStatsCalculatorFailures(session)) {
+                // log or fail for others, depeding on the session setting
                 log.error(e, "Error occurred when computing stats for query %s", session.getQueryId());
                 return PlanNodeStatsEstimate.unknown();
             }
             throw e;
         }
     }
 
+    boolean shouldIgnoreStatsCalculatorFailure(RuntimeException e)
+    {
+        if (e instanceof NotInTransactionException) {
+            // it is common to get NotInTransactionException if query gets cancelled during planning
+            return true;
+        }
+        return false;
+    }
+
     private PlanNodeStatsEstimate getGroupStats(GroupReference groupReference)
     {
         int group = groupReference.getGroupId();