Skip to content

Conversation

@BiteTheDDDDt
Copy link
Contributor

Proposed changes

use Allocator_ to manage memory on random_bytes/space

@BiteTheDDDDt
Copy link
Contributor Author

run buildall

@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

Copy link
Contributor

@xinyiZzz xinyiZzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions
Copy link
Contributor

PR approved by at least one committer and no changes requested.

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Jul 17, 2024
@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@BiteTheDDDDt
Copy link
Contributor Author

run cloud_p1

@yiguolei yiguolei merged commit ee9dc65 into apache:master Jul 17, 2024
dataroaring pushed a commit that referenced this pull request Jul 19, 2024
…#37997)

## Proposed changes
use Allocator_ to manage memory on random_bytes/space
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. dev/3.0.1-merged reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants