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.

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

Feature permission #142

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

123-komal
Copy link

No description provided.

wyzhang and others added 8 commits October 29, 2021 23:24
Detail:
Adding an option to specify number of concurrent threads
for sequential read operation
Detail:
Sample usage
java -cp ... \
--benchmarks=fillseq,readseq \
--num_concurrent_threads=1 \
--compress=false

where classpath can be found via
mvn dependency:resolve dependency:build-classpath
Detail:
Sample usage
java -cp ... \
--benchmarks=fillseq,readseq \
--num_concurrent_threads=1 \
--compress=false
--block_size=8096

where classpath can be found via
mvn dependency:resolve dependency:build-classpath
…MetaData

Fix an issue where a large byte array may get pinned down in memory
which could lead to out of memory.

Why:
FileMetadata stores smallest and largest key internally to identity
the start and end key range in a given file. The key is essentially
a slice of a byte array storing a key-value entry. This causes
the byte array to be pinned down in memory, which can be an issue
when the value is fairly large. Eventually this can lead to
out of memory.

What:
The change is to make a copy of the key (which should be typically
small) into a new byte array so the original byte array
(which can be large) can be freed.
Update Github Actions and checkstyle
[Security Vulnerability] Run build with unit tests without elevated permissions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants