Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[C++][Parquet] Minor enhancement for encryption #40731

Closed
mapleFU opened this issue Mar 22, 2024 · 1 comment
Closed

[C++][Parquet] Minor enhancement for encryption #40731

mapleFU opened this issue Mar 22, 2024 · 1 comment
Assignees
@mapleFU
Labels
Component: C++ Component: Parquet Type: enhancement
Milestone
16.0.0

Comments

@mapleFU
Copy link
Member

mapleFU commented Mar 22, 2024

Describe the enhancement requested

Applying some minor code enhancement

Component(s)

C++, Parquet
@github-actions github-actions bot mentioned this issue Mar 22, 2024
Merged
pitrou pushed a commit that referenced this issue Mar 26, 2024
@mapleFU
GH-40731: [C++][Parquet] Minor enhancement code of encryption (#40732)
145c4d3 
### Rationale for this change

* Applying more `std::move` in encryption
* Trying to use `NULLPTR` rather than `NULL`.

### What changes are included in this PR?

Applying more `std::move` in encryption

### Are these changes tested?

No need

### Are there any user-facing changes?

No

* GitHub Issue: #40731

Authored-by: mwish <maplewish117@gmail.com>
Signed-off-by: Antoine Pitrou <antoine@python.org>
@pitrou
Copy link
Member

pitrou commented Mar 26, 2024

Issue resolved by pull request 40732
#40732

@pitrou pitrou added this to the 16.0.0 milestone Mar 26, 2024
@pitrou pitrou closed this as completed Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mapleFU mapleFU

Labels
Component: C++ Component: Parquet Type: enhancement
Projects
None yet
Milestone
16.0.0
Development

No branches or pull requests
2 participants
@pitrou @mapleFU