Include ldb_tools and sst_dump_tools libraries in shared library

Summary: Ldb and sst_dump are not included in shared library now. Add it.

Test Plan:
Build
make release
make shared_lib

Reviewers: igor, kradhakrishnan, rven, yhchiang, IslamAbdelRahman, anthony

Reviewed By: IslamAbdelRahman, anthony

Subscribers: leveldb, dhruba

Differential Revision: https://reviews.facebook.net/D51735

Fix fb-only build for gcc 4.8.1

Summary: fb build for gcc 4.8.1 is broken for the ZSTD dependency is not fixed after ea11923550802a3eeebe81f2cc4a9a44cda32121. Fixing it.

Test Plan: ROCKSDB_FBCODE_BUILD_WITH_481=1 make -j40 OPT=-g

Subscribers: leveldb, dhruba

Differential Revision: https://reviews.facebook.net/D51741

Upgrade to ZSTD 0.4.2

Summary: Change to call the new compression function.

Test Plan: build and run db_bench with the compression to make sure it compresses.

Reviewers: anthony, rven, kradhakrishnan, IslamAbdelRahman, igor, yhchiang

Reviewed By: yhchiang

Subscribers: leveldb, dhruba

Differential Revision: https://reviews.facebook.net/D51603

Update HISTORY.md for RocksDB 4.1

Summary:
Update HISTORY.md for RocksDB 4.1

Test Plan:
no code change

Reviewers: igor, sdong, anthony, IslamAbdelRahman, rven, kradhakrishnan

Subscribers: leveldb

Fix Java Makefile

Summary:
In case rocksdb java package is built using make rocksdbjavastaticrelease, then
only those rocksdb binary built under the virtual environments is release build.

This patch fix this issue.

Test Plan:
PORTABLE=1 V=2 make rocksdbjavastaticrelease -j32
and make sure -O2 and -NDEBUG is included when compiling all source files.

Reviewers: sdong, anthony, IslamAbdelRahman, rven, kradhakrishnan, igor

Reviewed By: igor

Subscribers: dhruba, leveldb

Differential Revision: https://reviews.facebook.net/D50895

Merge pull request #821 from yuslepukhin/continue_windows_warnings

Enable C4200, C4702, C4305 windows warnings

Revert "Fix TSAN build for fbcode"

Summary:
Reverting c745f1d2c4b72a58874605c5da4180a4cb9a0366 because it
was based on an incorrect understanding of the correct way to enable
TSAN tests (it assumes "make COMPILE_WITH_TSAN=1 check" but in fact only
"COMPILE_WITH_TSAN=1 make check" is supported).

Test Plan: COMPILE_WITH_TSAN=1 make check

Reviewers: kradhakrishnan

Reviewed By: kradhakrishnan

Subscribers: dhruba

Differential Revision: https://reviews.facebook.net/D50445

Correct the comment of GetApproximateMemoryUsageByType

Summary: Correct the comment of GetApproximateMemoryUsageByType.

Test Plan: No code change.

Reviewers: igor, sdong, anthony, IslamAbdelRahman

Subscribers: dhruba, leveldb

Differential Revision: https://reviews.facebook.net/D50409

Revert "Adding new table properties"

Summary:
Reverting https://reviews.facebook.net/D34269 for now
after I landed it a flaky test started continuously failing, I am almost sure this patch is not related to the test but I will revert it until I figure out why it's failing

Test Plan: make check

Reviewers: kradhakrishnan

Subscribers: dhruba

Differential Revision: https://reviews.facebook.net/D50385

Enable C4305 'identifier' : truncation from 'type1' to 'type2'

Enable C4702 unreachable code

Enable C4200 warning
  nonstandard extension used : zero-sized array in struct/union

Merge pull request #820 from yuslepukhin/enable_compiler_warnings

Enable Windows warnings C4307 C4309 C4512 C4701