rocksdb icon indicating copy to clipboard operation
rocksdb copied to clipboard

Improves compaction output file cut to reduce WA

Open jay-zhuang opened this issue 1 year ago • 1 comments

Try cut the file at grandparent files boundaries, which could be the beginning or the ending. Test shows it can improve WA more than 10%.

Also includes #1963 to cut file before skippable grandparent file. Which is for use case like user adding 2 or more non-overlapping data range at the same time, it can reduce the overlapping of 2 datasets in the lower levels.

jay-zhuang avatar Sep 09 '22 03:09 jay-zhuang

@jay-zhuang has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot avatar Sep 15 '22 19:09 facebook-github-bot

@jay-zhuang has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar Sep 23 '22 19:09 facebook-github-bot

@jay-zhuang has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar Sep 23 '22 19:09 facebook-github-bot

@jay-zhuang has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar Sep 23 '22 21:09 facebook-github-bot

@jay-zhuang has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar Sep 23 '22 23:09 facebook-github-bot

@jay-zhuang has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar Sep 24 '22 01:09 facebook-github-bot

@jay-zhuang has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar Sep 26 '22 23:09 facebook-github-bot

@jay-zhuang has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar Sep 27 '22 03:09 facebook-github-bot

@jay-zhuang has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot avatar Sep 27 '22 03:09 facebook-github-bot

This change seems like it can have a big impact but the commit message content is too brief. Please include:

  • a few more sentences about what was changed
  • more detail on the perf test
  • a description of whether or not this behavior depends on options. From the files I see there is a new option, that option should be explained in the commit message

mdcallag avatar Sep 28 '22 15:09 mdcallag

@jay-zhuang has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar Sep 29 '22 00:09 facebook-github-bot

@jay-zhuang has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar Sep 30 '22 00:09 facebook-github-bot

@jay-zhuang has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot avatar Sep 30 '22 00:09 facebook-github-bot