pitrou commented on code in PR #43601:
URL: https://github.com/apache/arrow/pull/43601#discussion_r1805037473


##########
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:
   It seems it can occur sporadically even in non-stress tests:
   
https://github.com/ursacomputing/crossbow/actions/runs/11388191999/job/31684559075#step:7:3791
   ```
   [ RUN      ] TestS3FS.OpenOutputStreamMetadata
   /arrow/cpp/src/arrow/filesystem/s3fs_test.cc:593: Failure
   Failed
   'output->Close()' failed with IOError: When completing multiple part upload 
for key 'mdfile1' in bucket 'bucket': AWS Error NO_SUCH_UPLOAD during 
CompleteMultipartUpload operation: The specified multipart upload does not 
exist. The upload ID may be invalid, or the upload may have been aborted or 
completed.
   /arrow/cpp/src/arrow/filesystem/s3fs.cc:1830  
CleanupIfFailed(FinishPartUploadAfterFlush())
   [  FAILED  ] TestS3FS.OpenOutputStreamMetadata (215 ms)
   ```
   
   @kou Do you know if we were seeing this error before switching to HTTPS for 
Minio-based tests?



-- 
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]

Reply via email to