You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2020/10/09 18:17:10 UTC
[GitHub] [geode-native] albertogpz opened a new pull request #671: Feature/geode 8318
albertogpz opened a new pull request #671:
URL: https://github.com/apache/geode-native/pull/671
This pull request contains the work performed in the feature/asio branch to replace ACE_Sock with boost::asio and continues with some fixes on integration-test cases that timed out and the addition of asynchronous name resolution (allows to use the connect-timeout in name resolution too).
Note that this is still work in progress. Some integration-test cases fail, the integration/test cases for SSL are failing and there is cleaning up to be done, in the exception handling in the TcpConn class and several other places.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [geode-native] albertogpz commented on pull request #671: Feature/geode 8318
Posted by GitBox <gi...@apache.org>.
albertogpz commented on pull request #671:
URL: https://github.com/apache/geode-native/pull/671#issuecomment-729848739
This PR has been abandoned.
A better solution has been included in a new pull request.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [geode-native] albertogpz commented on pull request #671: Feature/geode 8318
Posted by GitBox <gi...@apache.org>.
albertogpz commented on pull request #671:
URL: https://github.com/apache/geode-native/pull/671#issuecomment-729822706
> This is a big diff, can you instead submit a pull request against the asio branch? We can concentrate on merging it into develop from there.
This PR has been abandoned. In the e-mail I sent I mentioned a new one: https://github.com/apache/geode-native/pull/695
Nevertheless I have created the pull request as you suggest against the asio branch: https://github.com/apache/geode-native/pull/697
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [geode-native] albertogpz closed pull request #671: Feature/geode 8318
Posted by GitBox <gi...@apache.org>.
albertogpz closed pull request #671:
URL: https://github.com/apache/geode-native/pull/671
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [geode-native] mreddington commented on pull request #671: Feature/geode 8318
Posted by GitBox <gi...@apache.org>.
mreddington commented on pull request #671:
URL: https://github.com/apache/geode-native/pull/671#issuecomment-729231461
This is a big diff, can you instead submit a pull request against the asio branch? We can concentrate on merging it into develop from there.
----------------------------------------------------------------
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:
users@infra.apache.org