zhztheplayer opened a new issue, #6748:
URL: https://github.com/apache/incubator-gluten/issues/6748

   ### Backend
   
   VL (Velox)
   
   ### Bug description
   
   Columnar rule appliers adopts `AdaptiveContext` which relies on an empirical 
value (introduced in https://github.com/apache/incubator-gluten/pull/1127) to 
look up on the current stack trace to determine whether it's in AQE planning 
([example](https://github.com/apache/incubator-gluten/blob/531e82a5bb7c93603661f4e0c5d98317d58cf109/gluten-core/src/main/scala/org/apache/gluten/extension/columnar/heuristic/HeuristicApplier.scala#L43-L48)).
 This method is very fragile given that different Spark version (including 
user-customized versions) may have different AQE call stacks.
   
   We'd make the code more robust perhaps by do a search on stack frames rather 
than using a empirical magic number. And eventually we should remove this 
in-AQE check since this is basically caller-sensitive code which impacts 
portability and encapsulation.
   
   ### Spark version
   
   None
   
   ### Spark configurations
   
   _No response_
   
   ### System information
   
   _No response_
   
   ### Relevant logs
   
   _No response_


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to