You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@gluten.apache.org by "zml1206 (via GitHub)" <gi...@apache.org> on 2025/01/23 01:30:31 UTC

[I] SparkPlan without logicalLink does not show correct fallback reason on UI [incubator-gluten]

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

   ### Backend
   
   VL (Velox)
   
   ### Bug description
   
   Currently, set the fallback reason to logical plan
   https://github.com/apache/incubator-gluten/blob/0f4489aeccb691abc2f739b7d7a1f4cb0037c093/gluten-substrait/src/main/scala/org/apache/spark/sql/execution/GlutenFallbackReporter.scala#L54-L69.
   There is a problem with this,  SparkPlan without logicalLink does not show correct fallback reason on UI. 
   Some additional nodes may be generated in gluten's rules, such as project generated by PushDownInputFileExpression.
   
   ### Spark version
   
   None
   
   ### Spark configurations
   
   _No response_
   
   ### System information
   
   _No response_
   
   ### Relevant logs
   
   ```bash
   
   ```


-- 
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: commits-unsubscribe@gluten.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@gluten.apache.org
For additional commands, e-mail: commits-help@gluten.apache.org


Re: [I] SparkPlan without logicalLink does not show correct fallback reason on UI [incubator-gluten]

Posted by "ulysses-you (via GitHub)" <gi...@apache.org>.
ulysses-you commented on issue #8596:
URL: https://github.com/apache/incubator-gluten/issues/8596#issuecomment-2608803101

   cc @wForget I remember you asked similar quetion before. do you have any update ?


-- 
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: commits-unsubscribe@gluten.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@gluten.apache.org
For additional commands, e-mail: commits-help@gluten.apache.org


Re: [I] SparkPlan without logicalLink does not show correct fallback reason on UI [incubator-gluten]

Posted by "zml1206 (via GitHub)" <gi...@apache.org>.
zml1206 commented on issue #8596:
URL: https://github.com/apache/incubator-gluten/issues/8596#issuecomment-2608774768

   @ulysses-you AQE will also trigger the gluten rule again and retag for new physical plan. Is it necessary to set the fallback reason to logical plan, if so, can you explain the reason in detail, thank you?


-- 
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: commits-unsubscribe@gluten.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@gluten.apache.org
For additional commands, e-mail: commits-help@gluten.apache.org