Skip to content

Commit ceab158

Browse files
committed
Fixed TLS-enabled build + cleanup
1 parent cfb9a12 commit ceab158

File tree

3 files changed

+33
-35
lines changed

3 files changed

+33
-35
lines changed

clickhouse/CMakeLists.txt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
SET ( clickhouse-cpp-lib-src
2-
base/coded.cpp
32
base/compressed.cpp
43
base/input.cpp
54
base/output.cpp
65
base/platform.cpp
76
base/socket.cpp
7+
base/coded.cpp
88

99
columns/array.cpp
1010
columns/date.cpp
@@ -72,7 +72,6 @@ INSTALL(FILES query.h DESTINATION include/clickhouse/)
7272

7373
# base
7474
INSTALL(FILES base/buffer.h DESTINATION include/clickhouse/base/)
75-
INSTALL(FILES base/coded.h DESTINATION include/clickhouse/base/)
7675
INSTALL(FILES base/compressed.h DESTINATION include/clickhouse/base/)
7776
INSTALL(FILES base/input.h DESTINATION include/clickhouse/base/)
7877
INSTALL(FILES base/output.h DESTINATION include/clickhouse/base/)

clickhouse/client.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
#include "client.h"
22
#include "protocol.h"
33

4-
#include "base/coded.h"
54
#include "base/compressed.h"
65
#include "base/socket.h"
76
#include "base/wire_format.h"

ut/client_ut.cpp

Lines changed: 32 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -1053,37 +1053,37 @@ INSTANTIATE_TEST_CASE_P(
10531053
}
10541054
));
10551055

1056-
// Special test that require properly configured TLS-enabled version of CH running locally
1057-
INSTANTIATE_TEST_CASE_P(
1058-
LocalhostTLS_None, ReadonlyClientTest,
1059-
::testing::Values(std::tuple<ClientOptions, std::vector<std::string> > {
1060-
ClientOptions()
1061-
.SetHost("127.0.0.1")
1062-
.SetPort(9440)
1063-
.SetUser("default")
1064-
.SetPingBeforeQuery(true)
1065-
.SetCompressionMethod(CompressionMethod::None)
1066-
.SetSSLOptions(ClientOptions::SSLOptions()
1067-
.PathToCADirectory("./CA/")
1068-
.UseSNI(false)),
1069-
QUERIES
1070-
}
1071-
));
1072-
1073-
INSTANTIATE_TEST_CASE_P(
1074-
LocalhostTLS_LZ4, ReadonlyClientTest,
1075-
::testing::Values(std::tuple<ClientOptions, std::vector<std::string> > {
1076-
ClientOptions()
1077-
.SetHost("127.0.0.1")
1078-
.SetPort(9440)
1079-
.SetUser("default")
1080-
.SetPingBeforeQuery(true)
1081-
.SetCompressionMethod(CompressionMethod::LZ4)
1082-
.SetSSLOptions(ClientOptions::SSLOptions()
1083-
.PathToCADirectory("./CA/")
1084-
.UseSNI(false)),
1085-
QUERIES
1086-
}
1087-
));
1056+
//// Special test that require properly configured TLS-enabled version of CH running locally
1057+
//INSTANTIATE_TEST_CASE_P(
1058+
// LocalhostTLS_None, ReadonlyClientTest,
1059+
// ::testing::Values(std::tuple<ClientOptions, std::vector<std::string> > {
1060+
// ClientOptions()
1061+
// .SetHost("127.0.0.1")
1062+
// .SetPort(9440)
1063+
// .SetUser("default")
1064+
// .SetPingBeforeQuery(true)
1065+
// .SetCompressionMethod(CompressionMethod::None)
1066+
// .SetSSLOptions(ClientOptions::SSLOptions()
1067+
// .PathToCADirectory("./CA/")
1068+
// .UseSNI(false)),
1069+
// QUERIES
1070+
// }
1071+
//));
1072+
1073+
//INSTANTIATE_TEST_CASE_P(
1074+
// LocalhostTLS_LZ4, ReadonlyClientTest,
1075+
// ::testing::Values(std::tuple<ClientOptions, std::vector<std::string> > {
1076+
// ClientOptions()
1077+
// .SetHost("127.0.0.1")
1078+
// .SetPort(9440)
1079+
// .SetUser("default")
1080+
// .SetPingBeforeQuery(true)
1081+
// .SetCompressionMethod(CompressionMethod::LZ4)
1082+
// .SetSSLOptions(ClientOptions::SSLOptions()
1083+
// .PathToCADirectory("./CA/")
1084+
// .UseSNI(false)),
1085+
// QUERIES
1086+
// }
1087+
//));
10881088

10891089
#endif

0 commit comments

Comments
 (0)