Uh oh!
There was an error while loading. Please reload this page.
Add an example of embedding indexes inside a parquet file - #16395
Conversation
| // Compute distinct values, serialize & Base64‑encode | ||
| let distinct: HashSet<_> = values.iter().copied().collect(); | ||
| let serialized = distinct.iter().cloned().collect::<Vec<_>>().join("\n"); | ||
| let b64 = general_purpose::STANDARD_NO_PAD.encode(serialized.as_bytes()); |
There was a problem hiding this comment.
I think this writes the index into the footer itself (as an opaque string)
This has at least 2 downsides
- The footer metadata will be much larger / longer to parse
- A binary index must be converted to/from strings (as you are doing here with b64)
Is it possible to write the binary data directly into the parquet file?
Specifically, so then the metadata looks something like
// Find out where the current write position islet offset_to_index_in_file = file.current_position()
file.write_all(distinct_index)?;// now, finalize the file with the parquet metadata:let props = WriterProperties::builder().set_key_value_metadata(Some(vec![KeyValue::new("distinct_index_data".into(),
offset_to_index_in_file.to_string(),)])).build();I am not sure how easy this would be to do with the current API
There was a problem hiding this comment.
Very good point @alamb! Thank you.
I will try to find a better solution, i agree the following downsides.
This has at least 2 downsides
- The footer metadata will be much larger / longer to parse
- A binary index must be converted to/from strings (as you are doing here with b64)
There was a problem hiding this comment.
I tried today, but found it's hard for current API to support this, will try it again.
There was a problem hiding this comment.
Try to using low level API, but it only works when we disable page index, if we setting page index, it will follow up the real row group data, and it conflicts with our embedding indexes.
There was a problem hiding this comment.
The code is here:
// Licensed to the Apache Software Foundation (ASF) under one// or more contributor license agreements. See the NOTICE file// distributed with this work for additional information// regarding copyright ownership. The ASF licenses this file// to you under the Apache License, Version 2.0 (the// "License"); you may not use this file except in compliance// with the License. You may obtain a copy of the License at//// http://www.apache.org/licenses/LICENSE-2.0//// Unless required by applicable law or agreed to in writing,// software distributed under the License is distributed on an// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY// KIND, either express or implied. See the License for the// specific language governing permissions and limitations// under the License.//! Example: embedding a "distinct values" index in a Parquet file's metadata//!//! 1. Read existing Parquet files//! 2. Compute distinct values for a target column using DataFusion//! 3. Serialize the distinct index to bytes and write to the new Parquet file//! with these encoded bytes appended as a custom metadata entry//! 4. Read each new parquet file, extract and deserialize the index from footer//! 5. Use the distinct index to prune files when queryinguse arrow::array::{ArrayRef,StringArray};use arrow::record_batch::RecordBatch;use arrow_schema::{DataType,Field,Schema,SchemaRef};use async_trait::async_trait;use datafusion::catalog::{Session,TableProvider};use datafusion::common::{HashMap,HashSet,Result};use datafusion::datasource::listing::PartitionedFile;use datafusion::datasource::memory::DataSourceExec;use datafusion::datasource::physical_plan::{FileScanConfigBuilder,ParquetSource};use datafusion::datasource::TableType;use datafusion::execution::object_store::ObjectStoreUrl;use datafusion::logical_expr::{Operator,TableProviderFilterPushDown};use datafusion::parquet::arrow::ArrowSchemaConverter;use datafusion::parquet::data_type::{ByteArray,ByteArrayType};use datafusion::parquet::errors::ParquetError;use datafusion::parquet::file::metadata::KeyValue;use datafusion::parquet::file::properties::WriterProperties;use datafusion::parquet::file::reader::{FileReader,SerializedFileReader};use datafusion::parquet::file::writer::SerializedFileWriter;use datafusion::physical_plan::ExecutionPlan;use datafusion::prelude::*;use datafusion::scalar::ScalarValue;use futures::AsyncWriteExt;use std::fs::{create_dir_all, read_dir,File};use std::io::{Read,Seek,SeekFrom,Write};use std::path::{Path,PathBuf};use std::sync::Arc;use tempfile::TempDir;/// We should disable page index support in the Parquet reader/// when we ennable this feature, since we are using a custom index.////// Example creating parquet file that/// contains specialized indexes that/// are ignored by other readers////// ```text/// ┌──────────────────────┐/// │┌───────────────────┐ │/// ││ DataPage │ │ Standard Parquet/// │└───────────────────┘ │ Data / pages/// │┌───────────────────┐ │/// ││ DataPage │ │/// │└───────────────────┘ │/// │ ... │/// │ │/// │┌───────────────────┐ │/// ││ DataPage │ │/// │└───────────────────┘ │/// │┏━━━━━━━━━━━━━━━━━━━┓ │/// │┃ ┃ │ key/value metadata/// │┃ Special Index ┃◀┼──── that points at the/// │┃ ┃ │ │ special index/// │┗━━━━━━━━━━━━━━━━━━━┛ │/// │╔═══════════════════╗ │ │/// │║ ║ │/// │║ Parquet Footer ║ │ │ Footer includes/// │║ ║ ┼────── thrift-encoded/// │║ ║ │ ParquetMetadata/// │╚═══════════════════╝ │/// └──────────────────────┘////// Parquet File/// ```/// DistinctIndexTable is a custom TableProvider that reads Parquet files#[derive(Debug)]structDistinctIndexTable{schema:SchemaRef,index:HashMap<String,HashSet<String>>,dir:PathBuf,}implDistinctIndexTable{/// Scan a directory, read each file's footer metadata into a mapfntry_new(dir:implInto<PathBuf>,schema:SchemaRef) -> Result<Self>{let dir = dir.into();letmut index = HashMap::new();for entry inread_dir(&dir)? {let path = entry?.path();if path.extension().and_then(|s| s.to_str()) != Some("parquet"){continue;}let file_name = path.file_name().unwrap().to_string_lossy().to_string();let distinct_set = read_distinct_index(&path)?;println!("Read distinct index for {}: {:?}", file_name, distinct_set);
index.insert(file_name, distinct_set);}Ok(Self{ schema, index, dir })}}pubstructIndexedParquetWriter<W:Write + Seek>{writer:SerializedFileWriter<W>,}impl<W:Write + Seek + Send>IndexedParquetWriter<W>{pubfntry_new(sink:W,schema:Arc<Schema>,props:WriterProperties,) -> Result<Self>{let schema_desc = ArrowSchemaConverter::new().convert(schema.as_ref())?;let props_ptr = Arc::new(props);let writer =
SerializedFileWriter::new(sink, schema_desc.root_schema_ptr(), props_ptr)?;Ok(Self{ writer })}}constINDEX_MAGIC:&[u8] = b"IDX1";fnwrite_file_with_index(path:&Path,values:&[&str]) -> Result<()>{let field = Field::new("category",DataType::Utf8,false);let schema = Arc::new(Schema::new(vec![field.clone()]));let arr:ArrayRef = Arc::new(StringArray::from(values.to_vec()));let batch = RecordBatch::try_new(schema.clone(),vec![arr])?;let distinct:HashSet<_> = values.iter().copied().collect();let serialized = distinct.into_iter().collect::<Vec<_>>().join("\n");let index_bytes = serialized.into_bytes();let props = WriterProperties::builder().build();let file = File::create(path)?;letmut writer = IndexedParquetWriter::try_new(file, schema.clone(), props)?;{letmut rg_writer = writer.writer.next_row_group()?;letmut ser_col_writer = rg_writer
.next_column()?
.ok_or_else(|| ParquetError::General("No column writer".into()))?;let col_writer = ser_col_writer.typed::<ByteArrayType>();let values_bytes:Vec<ByteArray> = batch
.column(0).as_any().downcast_ref::<StringArray>().unwrap().iter().map(|opt| ByteArray::from(opt.unwrap())).collect();println!("Writing values: {:?}", values_bytes);
col_writer.write_batch(&values_bytes,None,None)?;
ser_col_writer.close()?;
rg_writer.close()?;}let offset = writer.writer.inner().seek(SeekFrom::Current(0))?;let index_len = index_bytes.len()asu64;
writer.writer.inner().write_all(b"IDX1")?;
writer.writer.inner().write_all(&index_len.to_le_bytes())?;
writer.writer.inner().write_all(&index_bytes)?;
writer.writer.append_key_value_metadata(KeyValue::new("distinct_index_offset".to_string(),
offset.to_string(),));
writer.writer.append_key_value_metadata(KeyValue::new("distinct_index_length".to_string(),
index_bytes.len().to_string(),));
writer.writer.close()?;println!("Finished writing file to {}", path.display());Ok(())}fnread_distinct_index(path:&Path) -> Result<HashSet<String>,ParquetError>{letmut file = File::open(path)?;let file_size = file.metadata()?.len();println!("Reading index from {} (size: {})",
path.display(),
file_size
);let reader = SerializedFileReader::new(file.try_clone()?)?;let meta = reader.metadata().file_metadata();let offset = meta
.key_value_metadata().and_then(|kvs| kvs.iter().find(|kv| kv.key == "distinct_index_offset")).and_then(|kv| kv.value.as_ref()).ok_or_else(|| ParquetError::General("Missing index offset".into()))?
.parse::<u64>().map_err(|e| ParquetError::General(e.to_string()))?;let length = meta
.key_value_metadata().and_then(|kvs| kvs.iter().find(|kv| kv.key == "distinct_index_length")).and_then(|kv| kv.value.as_ref()).ok_or_else(|| ParquetError::General("Missing index length".into()))?
.parse::<usize>().map_err(|e| ParquetError::General(e.to_string()))?;println!("Reading index at offset: {}, length: {}", offset, length);
file.seek(SeekFrom::Start(offset))?;letmut magic_buf = [0u8;4];
file.read_exact(&mut magic_buf)?;if&magic_buf != INDEX_MAGIC{returnErr(ParquetError::General("Invalid index magic".into()));}letmut len_buf = [0u8;8];
file.read_exact(&mut len_buf)?;let stored_len = u64::from_le_bytes(len_buf)asusize;if stored_len != length {returnErr(ParquetError::General("Index length mismatch".into()));}letmut index_buf = vec![0u8; length];
file.read_exact(&mut index_buf)?;let s =
String::from_utf8(index_buf).map_err(|e| ParquetError::General(e.to_string()))?;Ok(s.lines().map(|s| s.to_string()).collect())}/// Implement TableProvider for DistinctIndexTable, using the distinct index to prune files#[async_trait]implTableProviderforDistinctIndexTable{fnas_any(&self) -> &dyn std::any::Any{self}fnschema(&self) -> SchemaRef{self.schema.clone()}fntable_type(&self) -> TableType{TableType::Base}/// Prune files before reading: only keep files whose distinct set contains the filter valueasyncfnscan(&self,_ctx:&dynSession,_proj:Option<&Vec<usize>>,filters:&[Expr],_limit:Option<usize>,) -> Result<Arc<dynExecutionPlan>>{// Look for a single `category = 'X'` filterletmut target:Option<String> = None;if filters.len() == 1{ifletExpr::BinaryExpr(expr) = &filters[0]{if expr.op == Operator::Eq{iflet(Expr::Column(c),Expr::Literal(ScalarValue::Utf8(Some(v)), _),) = (&*expr.left,&*expr.right){if c.name == "category"{println!("Filtering for category: {v}");
target = Some(v.clone());}}}}}// Determine which files to scanlet keep:Vec<String> = self.index.iter().filter(|(_f, set)| target.as_ref().is_none_or(|v| set.contains(v))).map(|(f, _)| f.clone()).collect();println!("Pruned files: {:?}", keep.clone());// Build ParquetSource for kept fileslet url = ObjectStoreUrl::parse("file://")?;let source = Arc::new(ParquetSource::default());letmut builder = FileScanConfigBuilder::new(url,self.schema.clone(), source);for file in keep {let path = self.dir.join(&file);let len = std::fs::metadata(&path)?.len();
builder = builder.with_file(PartitionedFile::new(
path.to_str().unwrap().to_string(),
len,));}Ok(DataSourceExec::from_data_source(builder.build()))}fnsupports_filters_pushdown(&self,fs:&[&Expr],) -> Result<Vec<TableProviderFilterPushDown>>{// Mark as inexact since pruning is file‑granularOk(vec![TableProviderFilterPushDown::Inexact; fs.len()])}}#[tokio::main]asyncfnmain() -> Result<()>{// 1. Create temp dir and write 3 Parquet files with different category setslet tmp = TempDir::new()?;let dir = tmp.path();create_dir_all(dir)?;write_file_with_index(&dir.join("a.parquet"),&["foo","bar","foo"])?;write_file_with_index(&dir.join("b.parquet"),&["baz","qux"])?;write_file_with_index(&dir.join("c.parquet"),&["foo","quux","quux"])?;// 2. Register our custom TableProviderlet field = Field::new("category",DataType::Utf8,false);let schema_ref = Arc::new(Schema::new(vec![field]));let provider = Arc::new(DistinctIndexTable::try_new(dir, schema_ref.clone())?);let ctx = SessionContext::new();
ctx.register_table("t", provider)?;// 3. Run a query: only files containing 'foo' get scannedlet df = ctx.sql("SELECT * FROM t").await?;
df.show().await?;// 3. Run a query: only files containing 'foo' get scannedlet df = ctx.sql("SELECT * FROM t WHERE category = 'foo'").await?;
df.show().await?;Ok(())}There was a problem hiding this comment.
This looks super cool @zhuqi-lucas
Try to using low level API, but it only works when we disable page index, if we setting page index, it will follow up the real row group data, and it conflicts with our embedding indexes.
I don't fully understand this concern -- I would probably have to play around with it some more
Are you willing to update this PR with this new example? I have some ideas on the various APIs we could use (like we could potentially encapsulate the index writing some more)
We could also then file a ticket upstream i arrow-rs with a description of what wasn't working with page indexes
There was a problem hiding this comment.
Thank you @alamb , updated the code without page index using low level API, i will continue debugging the case that our self defined index with page index.
alamb
left a comment
There was a problem hiding this comment.
Thank you @zhuqi-lucas -- this is really neat. I left some thoughts -- let me know what you think
| // Note: we disable page index support here since we are using a custom index, it has conflicts when testing. | ||
| // TODO: Remove this when we have a better solution for custom indexes with page index support. | ||
| let source = Arc::new(ParquetSource::default().with_enable_page_index(false)); |
There was a problem hiding this comment.
Thank you @alamb Currently, i disable the page index for reading, so this example will succeed, but if we enable page index, it will fail due to:
- We are writing the self defined index just after the data.
- But it seems, the page index offset info will write to the same place.
- I can't find a solution until now, need some help.
Thanks!
zhuqi-lucas
commented
Jun 19, 2025
Thank you @alamb, I am currently using this arrow-rs branch before the code merge: The example print logs, it's good, thanks! Writing values:[ByteArray{data:"foo"},ByteArray{data:"bar"},ByteArray{data:"foo"}]Writing custom index at offset:68, length:7Finished writing file to /var/folders/q7/zjtv8rvx2hz0_t_rjjq8p9k00000gp/T/.tmp9zCIJt/a.parquetWriting values:[ByteArray{data:"baz"},ByteArray{data:"qux"}]Writing custom index at offset:68, length:7Finished writing file to /var/folders/q7/zjtv8rvx2hz0_t_rjjq8p9k00000gp/T/.tmp9zCIJt/b.parquetWriting values:[ByteArray{data:"foo"},ByteArray{data:"quux"},ByteArray{data:"quux"}]Writing custom index at offset:70, length:8Finished writing file to /var/folders/q7/zjtv8rvx2hz0_t_rjjq8p9k00000gp/T/.tmp9zCIJt/c.parquetReading index from /var/folders/q7/zjtv8rvx2hz0_t_rjjq8p9k00000gp/T/.tmp9zCIJt/a.parquet(size:363)Reading index at offset:68, length:7Read distinct index for a.parquet:"a.parquet"Reading index from /var/folders/q7/zjtv8rvx2hz0_t_rjjq8p9k00000gp/T/.tmp9zCIJt/b.parquet (size:363)Reading index at offset:68, length:7Read distinct index for b.parquet:"b.parquet"Reading index from /var/folders/q7/zjtv8rvx2hz0_t_rjjq8p9k00000gp/T/.tmp9zCIJt/c.parquet (size:368)Reading index at offset:70, length:8Read distinct index for c.parquet:"c.parquet"Filteringfor category: foo
Pruned files:["c.parquet","a.parquet"]
+----------+
| category |
+----------+
| foo |
| foo |
| foo |
+----------+ |
4e399a8 to
c344843Comparealamb
commented
Jun 21, 2025
FYI @XiangpengHao and @@JigaoLuo -- here is another example of the somewhat crazy things you can do with parquet |
alamb
commented
Jun 21, 2025
I agree with what @zhuqi-lucas says too The way I think about this is that the parquet file's footer contains pointers (offsets) to the actual data in the file. There is no requirement that the footer points to all bytes within the file There are other interesting things that can be done with this setup too (for example, concatenating parquet files together without having to re-encode the data (you can just copy the bytes around and rewrite the footer) |
zhuqi-lucas
commented
Jun 21, 2025
This is amazing @alamb ! Thanks!
|
alamb
commented
Jun 26, 2025
I have a PR up to upgrade to the necessary version of arrow here Hopefully once that merges we can polish this PR up (and make a big deal about it) |
zhuqi-lucas
commented
Jun 27, 2025
Thank you @alamb , i will polish the code after we changing to arrow/parquet 55.2.0!
|
zhuqi-lucas
commented
Jul 1, 2025
Updated the code with the merged PR: #16575 |
alamb
left a comment
There was a problem hiding this comment.
Thank you @zhuqi-lucas -- I started going through this PR again in detail
I renamed the example to align with the other parquet examples, and I added it to the list of examples.
I also took a pass through the comments.
I have run out of time today, but I'll finish it up first thing tomorrow and hopefully merge
Thank you again so much!
| // 1. Create temp dir and write 3 Parquet files with different category sets | ||
| let tmp = TempDir::new()?; | ||
| let dir = tmp.path(); | ||
| create_dir_all(dir)?; |
There was a problem hiding this comment.
BTW I also double checked that these files can be read by duckdb, and the had no problems:
D select*from read_parquet('/tmp/parquet_index_data/*');
┌──────────┐
│ category │
│ varchar │
├──────────┤
│ foo │
│ bar │
│ foo │
│ baz │
│ qux │
│ foo │
│ quux │
│ quux │
└──────────┘There was a problem hiding this comment.
Thank you @alamb for the crossing checking! This is very valuable!
zhuqi-lucas
commented
Jul 2, 2025
Thank you very much @alamb ! This looks pretty good. |
alamb
commented
Jul 2, 2025
Hi @zhuqi-lucas -- spent a while this morning going over this PR carefully -- it is great! I hope you don't mind but I made some substantial edits to try and make it read a bit better:
In my mind none of this was required but since I plan to make a Huge Deal (TM) about this example publically I figured spending some extra time polishing it would be worthwhle |
alamb
commented
Jul 2, 2025
I think it is now ready to merge, but it would probably be good for someone else to go over it one last time to make sure it is clear |
Thank you @alamb looks great to me!
Perfect for this change! |
Uh oh!
There was an error while loading. Please reload this page.
jcsherin
commented
Jul 2, 2025
@alamb The overview documentation is very clear and love the ASCII art. |
Co-authored-by: Sherin Jacob <jacob@protoship.io>
alamb
commented
Jul 3, 2025
This is so great -- now we just need to write up a blog post 🎣 Thanks again @zhuqi-lucas -- this is going to be great |
Uh oh!
There was an error while loading. Please reload this page.
Which issue does this PR close?
Rationale for this change
What changes are included in this PR?
Are these changes tested?
Are there any user-facing changes?