address review comments

core/src/main/scala/org/apache/spark/sql/rapids/tool/ToolUtils.scala

@@ -318,16 +318,16 @@ object SQLMetricsStats {
 }
 
 object IgnoreExecs {
-  val AdaptiveSparkPlan = "AdaptiveSparkPlan" // AdaptiveSparkPlan is not a real exec. It is
-  // a wrapper for the whole plan.
-  val CollectLimit = "CollectLimit" // Collect Limit replacement can be slower on the
-  // GPU. Disabled by default.
-  val ScanExistingRDD = "Scan ExistingRDD"
-  val ExecuteCreateViewCommand = "Execute CreateViewCommand"
-  val ExistingRDD = "ExistingRDD"
-  val LocalTableScan = "LocalTableScan"
-
-  def getallIgnoreExecs = Set(AdaptiveSparkPlan, CollectLimit, ScanExistingRDD,
+  // AdaptiveSparkPlan is not a real exec. It is a wrapper for the whole plan.
+  private val AdaptiveSparkPlan = "AdaptiveSparkPlan"
+  // Collect Limit replacement can be slower on the GPU. Disabled by default.
+  private val CollectLimit = "CollectLimit"
+  private val ScanExistingRDD = "Scan ExistingRDD"
+  private val ExecuteCreateViewCommand = "Execute CreateViewCommand"
+  private val ExistingRDD = "ExistingRDD"
+  private val LocalTableScan = "LocalTableScan"
+
+  def getAllIgnoreExecs: Set[String] = Set(AdaptiveSparkPlan, CollectLimit, ScanExistingRDD,
     ExecuteCreateViewCommand, ExistingRDD, LocalTableScan)
 }
 

core/src/main/scala/org/apache/spark/sql/rapids/tool/qualification/QualificationAppInfo.scala

@@ -587,7 +587,7 @@ class QualificationAppInfo(
         }.flatten
         topLevelExecs ++ childrenExecs
       }.collect {
-        case x if !IgnoreExecs.getallIgnoreExecs.contains(x.exec) => x.exec
+        case x if !IgnoreExecs.getAllIgnoreExecs.contains(x.exec) => x.exec
       }.toSet.mkString(";").trim.replaceAll("\n", "").replace(",", ":")