rocksdb icon indicating copy to clipboard operation
rocksdb copied to clipboard

Build and link libfolly with RocksDB

Open anand1976 opened this issue 2 years ago • 3 comments

The current integration with folly requires cherry-picking folly source files to include in RocksDB for external CI builds. Its not scaleable as we depend on more features in folly, such as coroutines. This PR adds a dependency from RocksDB to the folly library when USE_FOLLY or USE_COROUTINES are set. We build folly using the build scripts in third-party/folly, relying on it to download and build its dependencies. A new Makefile target, build_folly, is provided to make building folly easier.

A new option, USE_FOLLY_LITE is added to retain the old model of compiling selected folly sources with RocksDB. This might be useful for short-term development.

anand1976 avatar Jun 02 '22 23:06 anand1976

What version of gcc does folly require? I use Ubuntu 20.04 and 22.04 -- both are LTS (long term supported) but 22.04 has yet to be fully released. With 20.04 the default gcc is 9.4 and with 22.04 it is 11.2.

mdcallag avatar Jul 19 '22 22:07 mdcallag

What version of gcc does folly require? I use Ubuntu 20.04 and 22.04 -- both are LTS (long term supported) but 22.04 has yet to be fully released. With 20.04 the default gcc is 9.4 and with 22.04 it is 11.2.

@mdcallag Folly coroutine support requires gcc 10 or newer. But if you're not using coroutines, gcc 7+ would work.

anand1976 avatar Jul 20 '22 14:07 anand1976

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

facebook-github-bot avatar Aug 04 '22 15:08 facebook-github-bot

@anand1976 has updated the pull request. You must reimport the pull request before landing.

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

@anand1976 has updated the pull request. You must reimport the pull request before landing.

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

@anand1976 has updated the pull request. You must reimport the pull request before landing.

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

@anand1976 has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar Sep 06 '22 18:09 facebook-github-bot

@anand1976 has updated the pull request. You must reimport the pull request before landing.

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

@anand1976 has updated the pull request. You must reimport the pull request before landing.

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

@anand1976 has updated the pull request. You must reimport the pull request before landing.

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

@anand1976 has updated the pull request. You must reimport the pull request before landing.

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

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

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

@anand1976 has updated the pull request. You must reimport the pull request before landing.

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

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

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

@anand1976 has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot avatar Sep 12 '22 02:09 facebook-github-bot

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

facebook-github-bot avatar Sep 12 '22 02:09 facebook-github-bot