[
https://issues.apache.org/jira/browse/GEODE-8102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17172737#comment-17172737
]
ASF GitHub Bot commented on GEODE-8102:
---------------------------------------
pivotal-jbarrett commented on a change in pull request #630:
URL: https://github.com/apache/geode-native/pull/630#discussion_r466748307
##########
File path: cppcache/src/TcpConn.hpp
##########
@@ -65,75 +62,33 @@ class APACHE_GEODE_EXPORT TcpConn : public Connector {
virtual void createSocket(ACE_HANDLE sock);
+ virtual ssize_t doOperation(const SockOp& op, void* buff, size_t sendlen,
+ ACE_Time_Value& waitTime, size_t& readLen) const;
+
public:
- size_t m_chunkSize;
-
- static size_t getDefaultChunkSize() {
- // Attempt to set chunk size to nearest OS page size
- // for perf improvement
- auto pageSize = boost::interprocess::mapped_region::get_page_size();
- if (pageSize > 16000000) {
- return 16000000;
- } else if (pageSize > 0) {
- return pageSize + (16000000 / pageSize) * pageSize;
- }
-
- return 16000000;
- }
-
- TcpConn(const char* hostname, int32_t port,
+ TcpConn(const std::string& hostname, uint16_t port,
std::chrono::microseconds waitSeconds, int32_t maxBuffSizePool);
- TcpConn(const char* ipaddr, std::chrono::microseconds waitSeconds,
+
+ TcpConn(const std::string& address, std::chrono::microseconds waitSeconds,
int32_t maxBuffSizePool);
- virtual ~TcpConn() override { close(); }
+ ~TcpConn() override {}
Review comment:
Because close is a virtual function and calling virtual functions from
destructors is undefined.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Link and load OpenSSL library directly
> --------------------------------------
>
> Key: GEODE-8102
> URL: https://issues.apache.org/jira/browse/GEODE-8102
> Project: Geode
> Issue Type: Improvement
> Components: native client
> Reporter: Jacob Barrett
> Priority: Major
> Labels: pull-request-available
>
> Lazy load the OpenSSL library directly, through ACE_SSL, into the
> apache-geode library. Currently we lazy load cryptoImpl, which immediately
> loads OpenSSL. The original intent was to avoid having an immediate
> dependency on OpenSSL at a time when its availability was questionable. On
> unix like systems OpenSSL is almost always available since so many other
> components in the OS depend on it. This immediate load dependency will have
> little to no effect on those systems. On some unix like systems the
> experience will improve by not having a runtime dependency on an intermediate
> library, cryptoImpl, that may need special treatments, like LD_LIBRARY_PATH
> or RPATH changes. On Windows, where OpenSSL is an anomaly we can use MSVC's
> lazy loading feature to only load OpenSSL if SSL/TLS is configured. This
> significantly improves the experience on Windows with regards to the location
> of cryptoImpl when using .NET.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
