H-Plus-Time commented on PR #6157: URL: https://github.com/apache/arrow-rs/pull/6157#issuecomment-2257974378
Aspects that I suspect are just _bad_: - GetRange - not exactly a fitting name :shrug: - get_byte_ranges - no longer Vec<range> wrt get_bytes. Intended to avoid dealing with throwing out GetRange::Suffix before passing into ObjectStore::get_ranges / reimplimenting that. Aspects that need consensus: - which of the two options (suffix, or non-suffix) gets the `load` method name (i.e. which is the default), and what the non-default method name should be. - How explicit should switching on (or if it's the default, off) suffix requests be? -- 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]
