alamb opened a new issue, #11389:
URL: https://github.com/apache/datafusion/issues/11389

   ### Is your feature request related to a problem or challenge?
   
   Whatever we can do to make it faster/easier to test and compile datafusion 
is good, and this includes keeping the dependency chain small. Reducing the 
dependency chain is also a good way to reduce potential security attacks. 
   
   
   I noticed in https://github.com/apache/datafusion/pull/11383 that we were 
upgrading a package called termtree that is only used in a single test. 
   
   ### Describe the solution you'd like
   
   Remove the uses of `termtree` package 
   
   https://github.com/search?q=repo%3Aapache%2Fdatafusion%20termtree&type=code
   
   BTW you can run the test in question via:
   
   ```shell
   cargo test --package datafusion-physical-plan --lib
   ```
   
   ### Describe alternatives you've considered
   
   I think it could be replaced with a `Display` impl and a function that 
printed out the leaves at the appropriate level
   
   I don't think this requires deep database knowledge, just some rust skills 
   
   
   ### Additional context
   
   cc @avantgardnerio as I think you are the original author


-- 
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.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