Skip to content

Commit 750140b

Browse files
fix: listing, no manifest setup (#1122)
the PR solves cases where entire data has no manifest also fixes the prefix creation for listing
1 parent 5286782 commit 750140b

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

src/query/listing_table_builder.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ impl ListingTableBuilder {
100100
let mut all_resolve = Vec::new();
101101
for prefix in prefixes {
102102
let path = relative_path::RelativePathBuf::from(format!("{}/{}", &self.stream, prefix));
103-
storage.absolute_url(path.as_relative_path()).to_string();
103+
let prefix = storage.absolute_url(path.as_relative_path()).to_string();
104104
if let Some(pos) = prefix.rfind("minute") {
105105
let hour_prefix = &prefix[..pos];
106106
minute_resolve

src/query/stream_schema_provider.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -679,6 +679,10 @@ fn return_listing_time_filters(
679679
manifest_list: &[ManifestItem],
680680
time_filters: &mut Vec<PartialTimeFilter>,
681681
) -> Option<Vec<PartialTimeFilter>> {
682+
if manifest_list.is_empty() {
683+
return Some(time_filters.clone());
684+
}
685+
682686
// vec to hold timestamps for listing
683687
let mut vec_listing_timestamps = Vec::new();
684688

0 commit comments

Comments
 (0)