-
Notifications
You must be signed in to change notification settings - Fork 34
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Signed-off-by: Ji Bin <[email protected]>
- Loading branch information
Showing
7 changed files
with
151 additions
and
12 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
|
||
MILVUS_REPO="https://github.com/milvus-io/milvus.git" | ||
MILVUS_VERSION="v2.3.1" | ||
MILVUS_VERSION="v2.3.2" | ||
BUILD_PROXY= | ||
BUILD_FORCE=NO |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,34 @@ | ||
From b731a16cb656bfe964a6253e6d7e3a28c30045ac Mon Sep 17 00:00:00 2001 | ||
From: Ji Bin <[email protected]> | ||
Date: Sun, 5 Nov 2023 18:27:32 +0800 | ||
Subject: [PATCH] fix for gettid | ||
|
||
Signed-off-by: Ji Bin <[email protected]> | ||
--- | ||
include/knowhere/comp/thread_pool.h | 3 ++- | ||
1 file changed, 2 insertions(+), 1 deletion(-) | ||
|
||
diff --git a/include/knowhere/comp/thread_pool.h b/include/knowhere/comp/thread_pool.h | ||
index a00920d..60546f5 100644 | ||
--- a/include/knowhere/comp/thread_pool.h | ||
+++ b/include/knowhere/comp/thread_pool.h | ||
@@ -13,6 +13,7 @@ | ||
|
||
#include <omp.h> | ||
#include <sys/resource.h> | ||
+#include <sys/syscall.h> | ||
|
||
#include <cerrno> | ||
#include <cstring> | ||
@@ -35,7 +36,7 @@ class ThreadPool { | ||
std::thread | ||
newThread(folly::Func&& func) override { | ||
return folly::NamedThreadFactory::newThread([&, func = std::move(func)]() mutable { | ||
- if (setpriority(PRIO_PROCESS, gettid(), 19) != 0) { | ||
+ if (setpriority(PRIO_PROCESS, syscall(SYS_gettid), 19) != 0) { | ||
LOG_KNOWHERE_ERROR_ << "Failed to set priority of knowhere thread. Error is: " | ||
<< std::strerror(errno); | ||
} else { | ||
-- | ||
2.42.1 | ||
|
25 changes: 25 additions & 0 deletions
25
milvus_binary/patches/milvus-v2.3.2/0001-fix-for-gettid.patch
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
From 4e4d7cd34d61fd671560eefde1674f6ec76a59ff Mon Sep 17 00:00:00 2001 | ||
From: Ji Bin <[email protected]> | ||
Date: Sun, 5 Nov 2023 18:49:29 +0800 | ||
Subject: [PATCH 1/2] fix for gettid | ||
|
||
Signed-off-by: Ji Bin <[email protected]> | ||
--- | ||
internal/core/thirdparty/knowhere/CMakeLists.txt | 1 + | ||
1 file changed, 1 insertion(+) | ||
|
||
diff --git a/internal/core/thirdparty/knowhere/CMakeLists.txt b/internal/core/thirdparty/knowhere/CMakeLists.txt | ||
index cc0729a84..aad2b3de7 100644 | ||
--- a/internal/core/thirdparty/knowhere/CMakeLists.txt | ||
+++ b/internal/core/thirdparty/knowhere/CMakeLists.txt | ||
@@ -41,6 +41,7 @@ FetchContent_Declare( | ||
GIT_TAG ${KNOWHERE_VERSION} | ||
SOURCE_DIR ${CMAKE_CURRENT_BINARY_DIR}/knowhere-src | ||
BINARY_DIR ${CMAKE_CURRENT_BINARY_DIR}/knowhere-build | ||
+ PATCH_COMMAND patch -p1 < ${CMAKE_SOURCE_DIR}/../../../patches/knowhere-f4c1757.patch | ||
DOWNLOAD_DIR ${THIRDPARTY_DOWNLOAD_PATH} ) | ||
|
||
FetchContent_GetProperties( knowhere ) | ||
-- | ||
2.42.1 | ||
|
28 changes: 28 additions & 0 deletions
28
milvus_binary/patches/milvus-v2.3.2/0002-link-with-CoreServices-for-macos.patch
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
From 9ec041db457b54188d61c47740238eaab338d952 Mon Sep 17 00:00:00 2001 | ||
From: Ji Bin <[email protected]> | ||
Date: Sun, 5 Nov 2023 22:22:51 +0800 | ||
Subject: [PATCH 2/2] link with CoreServices for macos | ||
|
||
Signed-off-by: Ji Bin <[email protected]> | ||
--- | ||
internal/core/src/storage/azure-blob-storage/CMakeLists.txt | 5 +++++ | ||
1 file changed, 5 insertions(+) | ||
|
||
diff --git a/internal/core/src/storage/azure-blob-storage/CMakeLists.txt b/internal/core/src/storage/azure-blob-storage/CMakeLists.txt | ||
index 91c2cc347..4441a2fae 100644 | ||
--- a/internal/core/src/storage/azure-blob-storage/CMakeLists.txt | ||
+++ b/internal/core/src/storage/azure-blob-storage/CMakeLists.txt | ||
@@ -25,5 +25,10 @@ set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-unused-parameter -Wno-return-type - | ||
add_library(blob-chunk-manager SHARED AzureBlobChunkManager.cpp) | ||
target_link_libraries(blob-chunk-manager PRIVATE Azure::azure-identity Azure::azure-storage-blobs) | ||
|
||
+if (APPLE) | ||
+ find_library(CORESERVICES_LIBRARY CoreServices) | ||
+ target_link_libraries(blob-chunk-manager PRIVATE ${CORESERVICES_LIBRARY}) | ||
+endif (APPLE) | ||
+ | ||
install(TARGETS blob-chunk-manager DESTINATION "${CMAKE_INSTALL_LIBDIR}") | ||
|
||
-- | ||
2.42.1 | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -21,7 +21,7 @@ | |
import json | ||
import hashlib | ||
|
||
__version__ = '2.3.1' | ||
__version__ = '2.3.2' | ||
|
||
LOGGERS = {} | ||
|
||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters