[GitHub] [arrow-rs] wjones127 commented on a diff in pull request #3973: Add ObjectStore::list_with_offset (#3970)

["wjones127 (via GitHub)" <gi...@apache.org>]

wjones127 commented on code in PR #3973:
URL: https://github.com/apache/arrow-rs/pull/3973#discussion_r1151270890


##########
object_store/src/lib.rs:
##########
@@ -371,11 +371,33 @@ pub trait ObjectStore: std::fmt::Display + Send + Sync + Debug + 'static {
     ///
     /// Prefixes are evaluated on a path segment basis, i.e. `foo/bar/` is a prefix of `foo/bar/x` but not of
     /// `foo/bar_baz/x`.
+    ///
+    /// Note: the order of returned [`ObjectMeta`] is not guaranteed

Review Comment:
   I admit I totally forgot about this detail. Is this an artifact of the requests not being in a particular order? or that local filesystems make no guarantees about order? Or that the order isn't guaranteed to be consistent between object stores?
   
   It does seems like S3 and GCS (and even Azure) return in a particular order.



-- 
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-unsubscribe@arrow.apache.org

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