Skip to content

Commit

Permalink
fix azure sdk compile error and test it
Browse files Browse the repository at this point in the history
Signed-off-by: sunby <[email protected]>
  • Loading branch information
sunby committed Sep 10, 2024
1 parent d458be7 commit 1b91655
Show file tree
Hide file tree
Showing 7 changed files with 30 additions and 402 deletions.
2 changes: 2 additions & 0 deletions cpp/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ if (WITH_OPENDAL)
include(libopendal)
endif()

find_package(Azure REQUIRED)
find_package(Boost REQUIRED)
find_package(Arrow REQUIRED)
find_package(protobuf REQUIRED)
Expand All @@ -26,6 +27,7 @@ file(GLOB_RECURSE SRC_FILES src/*.cpp src/*.cc)
add_library(milvus-storage ${SRC_FILES})
target_include_directories(milvus-storage PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}/include/milvus-storage ${CMAKE_CURRENT_SOURCE_DIR}/src ${CMAKE_CURRENT_SOURCE_DIR}/test/include)
set(LINK_LIBS
Azure::azure-core
arrow::arrow
Boost::boost
protobuf::protobuf
Expand Down
2 changes: 2 additions & 0 deletions cpp/conanfile.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ class StorageConan(ConanFile):
"aws-sdk-cpp:text-to-speech": False,
"aws-sdk-cpp:transfer": False,
"arrow:with_s3": True,
"arrow:with_azure": True,
"arrow:filesystem_layer": True,
"arrow:dataset_modules": True,
"arrow:parquet": True,
Expand Down Expand Up @@ -80,6 +81,7 @@ def configure(self):

def requirements(self):
self.requires("boost/1.81.0")
# self.requires("azure-sdk-for-cpp/1.11.3")
self.requires("arrow/17.0.0")
self.requires("openssl/3.1.2")
self.requires("protobuf/3.21.4")
Expand Down
38 changes: 0 additions & 38 deletions cpp/include/milvus-storage/fs/AliyunCredentialsClient.h

This file was deleted.

33 changes: 0 additions & 33 deletions cpp/include/milvus-storage/fs/AliyunCredentialsProvider.h

This file was deleted.

160 changes: 0 additions & 160 deletions cpp/src/fs/AliyunCredentialsClient.cpp

This file was deleted.

Loading

0 comments on commit 1b91655

Please sign in to comment.