Fixing Windows build using CMake (#7854)

Summary:
Builds were not producing Windows binaries properly in 6.15 branch:

```
00:00:46.413 Tests run: 11, Failures: 0, Errors: 2, Skipped: 0, Time elapsed: 0.183 sec <<< FAILURE! - in org.rocksdb.EventListenerTest
00:00:46.414 testAllCallbacksInvocation(org.rocksdb.EventListenerTest)  Time elapsed: 0.012 sec  <<< ERROR!
00:00:46.414 java.lang.UnsatisfiedLinkError: org.rocksdb.test.TestableEventListener.invokeAllCallbacks(J)V
00:00:46.414 	at org.rocksdb.test.TestableEventListener.invokeAllCallbacks(Native Method)
00:00:46.414 	at org.rocksdb.test.TestableEventListener.invokeAllCallbacks(TestableEventListener.java:19)
00:00:46.414 	at org.rocksdb.EventListenerTest.testAllCallbacksInvocation(EventListenerTest.java:436)
```

```
00:00:41.497        "D:\j\workspace\RocksDB_Build_Windows\build\java\rocksdbjni_headers.vcxproj" (default target) (3) ->
00:00:41.497        (CustomBuild target) ->
00:00:41.497          CUSTOMBUILD : error : Could not find class file for 'org.rocksdb.TestableEventListener'. [D:\j\workspace\RocksDB_Build_Windows\build\java\rocksdbjni_headers.vcxproj]
```

Also failed on Linux as library was not initialized yet:

```
00:01:25.103 Running org.rocksdb.NativeComparatorWrapperTest
00:01:25.133 Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.006 sec <<< FAILURE! - in org.rocksdb.NativeComparatorWrapperTest
00:01:25.133 rountrip(org.rocksdb.NativeComparatorWrapperTest)  Time elapsed: 0.002 sec  <<< ERROR!
00:01:25.133 java.lang.UnsatisfiedLinkError: org.rocksdb.NativeComparatorWrapperTest$NativeStringComparatorWrapper.newStringComparator()J
00:01:25.133 	at org.rocksdb.NativeComparatorWrapperTest$NativeStringComparatorWrapper.newStringComparator(Native Method)
00:01:25.133 	at org.rocksdb.NativeComparatorWrapperTest$NativeStringComparatorWrapper.initializeNative(NativeComparatorWrapperTest.java:87)
00:01:25.133 	at org.rocksdb.RocksCallbackObject.<init>(RocksCallbackObject.java:28)
00:01:25.133 	at org.rocksdb.AbstractComparator.<init>(AbstractComparator.java:20)
00:01:25.133 	at org.rocksdb.NativeComparatorWrapper.<init>(NativeComparatorWrapper.java:16)
00:01:25.133 	at org.rocksdb.NativeComparatorWrapperTest$NativeStringComparatorWrapper.<init>(NativeComparatorWrapperTest.java:82)
00:01:25.133 	at org.rocksdb.NativeComparatorWrapperTest.rountrip(NativeComparatorWrapperTest.java:30)
```

Pull Request resolved: https://github.com/facebook/rocksdb/pull/7854

Reviewed By: jay-zhuang

Differential Revision: D25873378

Pulled By: ajkr

fbshipit-source-id: 88afb08bfd30edff31f17da063e636df0769cbfe
main
Tomas Kolda 4 years ago committed by Facebook GitHub Bot
parent 1001bc01c9
commit d76a8eeef7
  1. 5
      java/CMakeLists.txt
  2. 3
      java/src/test/java/org/rocksdb/NativeComparatorWrapperTest.java
  3. 3
      utilities/blob_db/blob_db_test.cc

@ -65,6 +65,7 @@ set(JNI_NATIVE_SOURCES
rocksjni/table.cc rocksjni/table.cc
rocksjni/table_filter.cc rocksjni/table_filter.cc
rocksjni/table_filter_jnicallback.cc rocksjni/table_filter_jnicallback.cc
rocksjni/testable_event_listener.cc
rocksjni/thread_status.cc rocksjni/thread_status.cc
rocksjni/trace_writer.cc rocksjni/trace_writer.cc
rocksjni/trace_writer_jnicallback.cc rocksjni/trace_writer_jnicallback.cc
@ -283,6 +284,7 @@ set(JAVA_TEST_CLASSES
src/test/java/org/rocksdb/WriteBatchTest.java src/test/java/org/rocksdb/WriteBatchTest.java
src/test/java/org/rocksdb/util/CapturingWriteBatchHandler.java src/test/java/org/rocksdb/util/CapturingWriteBatchHandler.java
src/test/java/org/rocksdb/util/WriteBatchGetter.java src/test/java/org/rocksdb/util/WriteBatchGetter.java
src/test/java/org/rocksdb/test/TestableEventListener.java
) )
include(FindJava) include(FindJava)
@ -434,6 +436,8 @@ if(${CMAKE_VERSION} VERSION_LESS "3.11.4" OR (${Java_VERSION_MINOR} STREQUAL "7"
org.rocksdb.CompactRangeOptions org.rocksdb.CompactRangeOptions
org.rocksdb.ComparatorOptions org.rocksdb.ComparatorOptions
org.rocksdb.CompressionOptions org.rocksdb.CompressionOptions
org.rocksdb.ConcurrentTaskLimiterImpl
org.rocksdb.ConfigOptions
org.rocksdb.DBOptions org.rocksdb.DBOptions
org.rocksdb.DirectSlice org.rocksdb.DirectSlice
org.rocksdb.Env org.rocksdb.Env
@ -502,6 +506,7 @@ if(${CMAKE_VERSION} VERSION_LESS "3.11.4" OR (${Java_VERSION_MINOR} STREQUAL "7"
org.rocksdb.WriteBatchTest org.rocksdb.WriteBatchTest
org.rocksdb.WriteBatchTestInternalHelper org.rocksdb.WriteBatchTestInternalHelper
org.rocksdb.WriteBufferManager org.rocksdb.WriteBufferManager
org.rocksdb.test.TestableEventListener
) )
create_javah( create_javah(

@ -15,6 +15,9 @@ import java.util.Comparator;
import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertEquals;
public class NativeComparatorWrapperTest { public class NativeComparatorWrapperTest {
static {
RocksDB.loadLibrary();
}
@Rule @Rule
public TemporaryFolder dbFolder = new TemporaryFolder(); public TemporaryFolder dbFolder = new TemporaryFolder();

@ -828,8 +828,7 @@ TEST_F(BlobDBTest, SstFileManagerRestart) {
const auto &fs = db_options.env->GetFileSystem(); const auto &fs = db_options.env->GetFileSystem();
ASSERT_OK(CreateFile(fs, blob_dir + "/000666.blob.trash", "", false)); ASSERT_OK(CreateFile(fs, blob_dir + "/000666.blob.trash", "", false));
ASSERT_OK(CreateFile(fs, blob_dir + "/000888.blob.trash", "", true)); ASSERT_OK(CreateFile(fs, blob_dir + "/000888.blob.trash", "", true));
ASSERT_OK( ASSERT_OK(CreateFile(fs, blob_dir + "/something_not_match.trash", "", false));
CreateFile(fs, blob_dir + "/something_not_match.trash", "", false));
// Make sure that reopening the DB rescan the existing trash files // Make sure that reopening the DB rescan the existing trash files
Open(bdb_options, db_options); Open(bdb_options, db_options);

Loading…
Cancel
Save