[PR #1100] [MERGED] Change test_concurrency test #1811

New issue

Closed

opened 2026-03-04 02:02:17 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 02:02:17 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/1100
Author: @ggtakec
Created: 7/15/2019
Status: ✅ Merged
Merged: 7/15/2019
Merged by: @gaul

Base: master ← Head: test_concurrency

📝 Commits (1)

3d63a25 Change test_concurrency test

📊 Changes

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

View changed files

📝 test/integration-test-main.sh (+6 -6)

📄 Description

Relevant Issue (if applicable)

n/a

Details

This PR corrected and revised the test test_concurrency.

Fixed the following part of test_concurrency test.

Number of test files
Initial 10 pieces, 100 pieces after writing. The new tests were initially set to 5 after writing.
Number of parallel executions
The number of parallel executions has been changed from 2 to 10.
Number of single processing loops
It has been changed from 100 to 5. This is the same as the number of test files.
File created in loop
We randomly created 1 to 100 files, but changed to 1 to 5 files.
Specify file name clearly
This is because FUSE may create a special temporary file (a file name such as .fuse_hidden0000005000000001), which may cause the test to fail.

If I misunderstood the nature of this test, I will not adapt this PR.
The new fix is to increase the number of parallels and reduce the number of files.
This has increased the chance of file collisions.

Also, increasing the number of parallels and reducing the number of single processing loops will reduce test time.
And the number of log lines at the time of error is also a quarter.

_{🔄 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/1100 **Author:** [@ggtakec](https://github.com/ggtakec) **Created:** 7/15/2019 **Status:** ✅ Merged **Merged:** 7/15/2019 **Merged by:** [@gaul](https://github.com/gaul) **Base:** `master` ← **Head:** `test_concurrency` --- ### 📝 Commits (1) - [`3d63a25`](https://github.com/s3fs-fuse/s3fs-fuse/commit/3d63a255a0f834cc3fa8477d689243d9fd7a566c) Change test_concurrency test ### 📊 Changes **1 file changed** (+6 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `test/integration-test-main.sh` (+6 -6) </details> ### 📄 Description ### Relevant Issue (if applicable) n/a ### Details This PR corrected and revised the test test_concurrency. Fixed the following part of test_concurrency test. - Number of test files Initial 10 pieces, 100 pieces after writing. The new tests were initially set to 5 after writing. - Number of parallel executions The number of parallel executions has been changed from 2 to 10. - Number of single processing loops It has been changed from 100 to 5. This is the same as the number of test files. - File created in loop We randomly created 1 to 100 files, but changed to 1 to 5 files. - Specify file name clearly This is because FUSE may create a special temporary file (a file name such as .fuse_hidden0000005000000001), which may cause the test to fail. If I misunderstood the nature of this test, I will not adapt this PR. The new fix is to increase the number of parallels and reduce the number of files. This has increased the chance of file collisions. Also, increasing the number of parallels and reducing the number of single processing loops will reduce test time. And the number of log lines at the time of error is also a quarter. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>