houqp commented on a change in pull request #1779:
URL: https://github.com/apache/arrow-datafusion/pull/1779#discussion_r801262929
##########
File path: datafusion/src/datasource/object_store/local.rs
##########
@@ -39,6 +39,11 @@ pub struct LocalFileSystem;
#[async_trait]
impl ObjectStore for LocalFileSystem {
async fn list_file(&self, prefix: &str) -> Result<FileMetaStream> {
+ let prefix = if let Some((_scheme, path)) = prefix.split_once("://") {
Review comment:
if objecstore instances are already sharded by host and port based on
changes in `get_by_uri`, shouldn't we only accept path of the uri in this
method? In other words, I think we can safely assume for a given object store
instance, all requests should be referencing the same port and host right?
--
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...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
