[PR #1586] [MERGED] Bypass an OSX test(no different from other OS) which takes very time consuming #2061

Closed
opened 2026-03-04 02:03:30 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/1586
Author: @ggtakec
Created: 2/23/2021
Status: Merged
Merged: 2/23/2021
Merged by: @gaul

Base: masterHead: bypass_test


📝 Commits (2)

  • 8ae9a83 Bypass an OSX test(no different from other OS) which takes very time consuming
  • da5a7c1 Merge branch 'master' into bypass_test

📊 Changes

1 file changed (+8 additions, -1 deletions)

View changed files

📝 test/integration-test-main.sh (+8 -1)

📄 Description

Relevant Issue (if applicable)

#1579 #1582

Details

test_update_metadata_external_large_object The test repeats uploading and downloading large objects several times.
It takes a long time to test, especially because it is not affected by the stats (noobj) cache.
Before adding this test, OSX took longer to run the test than other OSs, but after adding this test, it's up to 10 minutes slower.
The test itself is considered OS-independent and suggests bypassing in the case of OSX.
In addition, after improving the performance of s3fs itself, it is a condition to review this test and restore it.

I think it should be merged, but please give us your opinion.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/s3fs-fuse/s3fs-fuse/pull/1586 **Author:** [@ggtakec](https://github.com/ggtakec) **Created:** 2/23/2021 **Status:** ✅ Merged **Merged:** 2/23/2021 **Merged by:** [@gaul](https://github.com/gaul) **Base:** `master` ← **Head:** `bypass_test` --- ### 📝 Commits (2) - [`8ae9a83`](https://github.com/s3fs-fuse/s3fs-fuse/commit/8ae9a839512e04dbceeb00f6d6dc121bf5bba2ed) Bypass an OSX test(no different from other OS) which takes very time consuming - [`da5a7c1`](https://github.com/s3fs-fuse/s3fs-fuse/commit/da5a7c118b637ce0343160e157171e06d61944cf) Merge branch 'master' into bypass_test ### 📊 Changes **1 file changed** (+8 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `test/integration-test-main.sh` (+8 -1) </details> ### 📄 Description ### Relevant Issue (if applicable) #1579 #1582 ### Details test_update_metadata_external_large_object The test repeats uploading and downloading large objects several times. It takes a long time to test, especially because it is not affected by the stats (noobj) cache. Before adding this test, OSX took longer to run the test than other OSs, but after adding this test, it's up to 10 minutes slower. The test itself is considered OS-independent and suggests bypassing in the case of OSX. In addition, after improving the performance of s3fs itself, it is a condition to review this test and restore it. I think it should be merged, but please give us your opinion. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-04 02:03:30 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/s3fs-fuse#2061
No description provided.