ding-young commented on PR #16814: URL: https://github.com/apache/datafusion/pull/16814#issuecomment-3126700775
> Personally, I prefer to use `time -v dfbench xxxxx` to get memory information. The main advantage is that it doesn't require any extra compilation of dfbench. @UBarney Thanks for the feedback! I actually considered writing a Python script (+ time -v or reading /proc/stat~~) instead, for the same reason you mentioned. But I eventually chose this because I wanted to make sure that the utility user compiles the benchmark binary ahead of time and for the reason @2010YOUY01 mentioned. (although max rss will be our primary focus) Anyway, I agree the double compilation step is not so desirable. If you think it is too painful, I’m definitely open to alternative ideas — for example, rewriting `mem_profile.rs` as a Python script. -- 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
