pitrou commented on code in PR #43601:
URL: https://github.com/apache/arrow/pull/43601#discussion_r1805041631
##########
cpp/src/arrow/filesystem/s3fs_test.cc:
##########
@@ -897,7 +940,15 @@ TEST_F(TestS3FS, GetFileInfoGenerator) {
// Non-root dir case is tested by generic tests
}
-TEST_F(TestS3FS, GetFileInfoGeneratorStress) {
+class TestS3FSHTTP : public TestS3FS {
+ public:
+ void SetUp() override {
+ enable_tls_if_supported_ = false;
Review Comment:
I think this may have to do with forcing "localhost" for TLS certificate
check purposes: in some cases, a port number might be reused?
What we should perhaps do is:
1) switch all tests to HTTP _except_ the SSE customer key tests
2) use "localhost" only for HTTPS tests, otherwise use `GetListenAddress()`
without a host argument.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
