findepi commented on PR #17168:
URL: https://github.com/apache/datafusion/pull/17168#issuecomment-3183958911

   Maybe the "usage" is here, in a TODO comment
   
https://github.com/apache/datafusion/blob/541a67d12c3c94278facec3a1fc60fb959d4eb62/datafusion/expr/src/expr.rs#L3252
   
   We need to decide what to do.
   Keep display_name() code that's dead/redundant for a year now.
   Or proceed with removal. Then the above comment should be gone.
   
   As https://github.com/apache/datafusion/issues/11782, 
https://github.com/apache/datafusion/issues/10364#issuecomment-2288257503 note, 
having many "name" functions is hard to follow, so I'm inclined to proceed with 
removal.


-- 
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: github-unsubscr...@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to