reject at least one promise when fetching a row group fails (#208)

severo · web-flow · commit 818af038d423 · 2025-04-07T19:27:00.000+02:00
diff --git a/src/lib/tableProvider.ts b/src/lib/tableProvider.ts
@@ -75,7 +75,11 @@ export function parquetDataFrame(from: AsyncBufferFrom, metadata: FileMetaData):
           }
         })
         .catch((error: unknown) => {
-          console.error('Error fetching row group', error)
+          const prefix = `Error fetching row group ${groupIndex} (${rowStart}-${rowEnd}).`
+          console.error(prefix, error)
+          const reason = `${prefix} ${error}`
+          // reject the index of the first row (it's enough to trigger the error bar)
+          data[rowStart]?.index.reject(reason)
         })
       groups[groupIndex] = true
     }

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,11 @@ export function parquetDataFrame(from: AsyncBufferFrom, metadata: FileMetaData):`
`75`	`75`	`}`
`76`	`76`	`})`
`77`	`77`	`.catch((error: unknown) => {`
`78`		`- console.error('Error fetching row group', error)`
	`78`	+ const prefix = `Error fetching row group ${groupIndex} (${rowStart}-${rowEnd}).`
	`79`	`+ console.error(prefix, error)`
	`80`	+ const reason = `${prefix} ${error}`
	`81`	`+ // reject the index of the first row (it's enough to trigger the error bar)`
	`82`	`+ data[rowStart]?.index.reject(reason)`
`79`	`83`	`})`
`80`	`84`	`groups[groupIndex] = true`
`81`	`85`	`}`