Can't drop SortExecTransformer in convert SortAggregateExec to HashAggregateExecBaseTransformer when ras enbled

[zml1206]

Backend

VL (Velox)

Bug description

withSQLConf(
      ("spark.sql.adaptive.enabled", "false"),
      ("spark.sql.test.forceApplySortAggregate", "true"),
      ("spark.gluten.sql.columnar.forceShuffledHashJoin", "true")) {
      createTPCHNotNullTables()
     spark.sql("select l_partkey,count(1) from lineitem group by l_partkey").explain
}

== Physical Plan ==

VeloxColumnarToRowExec
+- ^(2) HashAggregateTransformer(keys=[l_partkey#77L], functions=[count(1)], output=[l_partkey#77L, count(1)#123L])
   +- ^(2) SortExecTransformer [l_partkey#77L ASC NULLS FIRST], false, 0
      +- ^(2) InputIteratorTransformer[l_partkey#77L, count#127L]
         +- ^(2) InputAdapter
            +- ^(2) RowToVeloxColumnar
               +- ^(2) Exchange hashpartitioning(l_partkey#77L, 5), ENSURE_REQUIREMENTS, [plan_id=466]
                  +- ^(2) VeloxColumnarToRowExec
                     +- ^(1) FlushableHashAggregateTransformer(keys=[l_partkey#77L], functions=[partial_count(1)], output=[l_partkey#77L, count#127L])
                        +- ^(1) SortExecTransformer [l_partkey#77L ASC NULLS FIRST], false, 0
                           +- ^(1) NativeFileScan parquet [l_partkey#77L] Batched: true, DataFilters: [], Format: Parquet, Location: InMemoryFileIndex(1 paths)[file:/Users/zml/Desktop/git_hub/incubator-gluten/backends-velox/target..., PartitionFilters: [], PushedFilters: [], ReadSchema: struct<l_partkey:bigint>

Spark version

None

Spark configurations

No response

System information

No response

Relevant logs

No response

[zml1206]

Is this a known issue? @zhztheplayer

[zhztheplayer]

This is a known issue. Based on current development status of RAS, one of the best solution is to have an individual rule to match on Sort + Hash Agg to remove that sort.

[zhztheplayer]

If it's about the golden check of RAS, you can just put whatever the suite generates as new golden file in PR and ping me with a comment. I would take care of that part later on.

[zml1206]

If it's about the golden check of RAS, you can just put whatever the suite generates as new golden file in PR and ping me with a comment. I would take care of that part later on.

OK, thank you.