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

Fix candidate file comparison when using path ids #251

Merged
merged 1 commit into from
Aug 31, 2014

Conversation

nbougalis
Copy link
Contributor

The function CompareCandidateFile incorrectly compares a file's path_id against itself and will always return false.

This is my first pull request against rocksdb and I have completed the CLA.

@igorcanadi
Copy link
Collaborator

lol, nice catch :)

igorcanadi added a commit that referenced this pull request Aug 31, 2014
Fix candidate file comparison when using path ids
@igorcanadi igorcanadi merged commit 985a31c into facebook:master Aug 31, 2014
@vinniefalco
Copy link

Cheater...did you run a source code analysis tool?

hunterlxt pushed a commit to hunterlxt/rocksdb that referenced this pull request Aug 27, 2021
* add option to disable write stall

Signed-off-by: Connor1996 <zbk602423539@gmail.com>
Nazgolze pushed a commit to Nazgolze/rocksdb-1 that referenced this pull request Sep 21, 2021
…ld-3.0.0

chore(package): update prebuild to version 3.0.0
Little-Wallace pushed a commit to Little-Wallace/rocksdb that referenced this pull request Nov 11, 2021
* add option to disable write stall

Signed-off-by: Connor1996 <zbk602423539@gmail.com>
Little-Wallace pushed a commit to Little-Wallace/rocksdb that referenced this pull request Dec 24, 2021
* add option to disable write stall

Signed-off-by: Connor1996 <zbk602423539@gmail.com>
Signed-off-by: tabokie <xy.tao@outlook.com>

Co-authored-by: Connor <zbk602423539@gmail.com>
Little-Wallace pushed a commit to Little-Wallace/rocksdb that referenced this pull request Dec 24, 2021
… version (facebook#258)

* Revert "Add an option to disable write stall (facebook#251) (facebook#252)"

This reverts commit 32b29a1.

Signed-off-by: tabokie <xy.tao@outlook.com>

* use ignore_unknown_options regardless of RocksDB version

Signed-off-by: tabokie <xy.tao@outlook.com>

* disable only for lower version and fix test

Signed-off-by: tabokie <xy.tao@outlook.com>
BusyJay pushed a commit to BusyJay/rocksdb that referenced this pull request Jul 25, 2022
* add option to disable write stall

Signed-off-by: Connor1996 <zbk602423539@gmail.com>
Signed-off-by: tabokie <xy.tao@outlook.com>
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.

None yet

3 participants