alamb commented on PR #195: URL: https://github.com/apache/parquet-site/pull/195#issuecomment-5333361259
> Thanks @alamb, it reads very nicely. > > One nit - should we merge four images of perf charts into one? I think matplotlib [supports](https://matplotlib.org/stable/gallery/subplots_axes_and_figures/subplots_demo.html) that and it would look more uniform. it sounds like a good idea to me > > About the perf charts contents/prose around it - should we add byte stream split to the bench or remove it from the text? I am not sure -- I want to keep the focus here on ALP rather than all the other potential possibilities Like I am thinking "what conclusion would we want a reader to take away from a chart with byte_stream_split?" Is the point that it is still the best compressing of the floating point options? That sort of takes away from the high level message of the blog I think Maybe we could add it to the benchmark repo but not the final summary charts on the blog? -- 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]
