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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to