Skip to content

Conversation

@mapleFU
Copy link
Member

@mapleFU mapleFU commented Sep 26, 2025

Which issue does this PR close?

Rationale for this change

See the issue

What changes are included in this PR?

&mut -> &

Are these changes tested?

Covered by existing

Are there any user-facing changes?

No

@github-actions github-actions bot added the parquet Changes to the parquet crate label Sep 26, 2025
@mapleFU mapleFU force-pushed the remove-mut-for-get-bf-fns branch from 4e4f0fc to d2cf38f Compare September 26, 2025 13:21
@mbrobbel mbrobbel changed the title [Parquet] Minor: Remove mut ref for geting row-group bloom filter [Parquet] Minor: Remove mut ref for getting row-group bloom filter Sep 26, 2025
@alamb alamb merged commit 6739996 into apache:main Sep 26, 2025
17 checks passed
@alamb
Copy link
Contributor

alamb commented Sep 26, 2025

🚀 thanks @mapleFU and @mapleFU

@mapleFU
Copy link
Member Author

mapleFU commented Sep 26, 2025

( I was at twice, lol )

@mapleFU mapleFU deleted the remove-mut-for-get-bf-fns branch September 26, 2025 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

parquet Changes to the parquet crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Parquet] Remove useless mut requirements in geting bloom filter function

3 participants