[GitHub] [arrow] mapleFU commented on issue #33683: [C++][Parquet] Would Arrow::FileReader support filter evaluating and optimize by BloomFilter

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

mapleFU commented on issue #33683:
URL: https://github.com/apache/arrow/issues/33683#issuecomment-1572055377

   Sorry for late reply because I'm a bit busy these days. I found a problem that bloom filter is not trival, it might enhance the performance, and might not. Should I add an `use_bloom_filter` options in `ParquetFragmentScanOptions` ?
   
   ```c++
   /// \brief Per-scan options for Parquet fragments
   class ARROW_DS_EXPORT ParquetFragmentScanOptions : public FragmentScanOptions {
    public:
     ParquetFragmentScanOptions();
     std::string type_name() const override { return kParquetTypeName; }
   
     /// Reader properties. Not all properties are respected: memory_pool comes from
     /// ScanOptions.
     std::shared_ptr<parquet::ReaderProperties> reader_properties;
     /// Arrow reader properties. Not all properties are respected: batch_size comes from
     /// ScanOptions. Additionally, dictionary columns come from
     /// ParquetFileFormat::ReaderOptions::dict_columns.
     std::shared_ptr<parquet::ArrowReaderProperties> arrow_reader_properties;
   };
   ```
   
   @westonpace 


-- 
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