Commit 9d5b3ff
Write, List, Rename & Delete Microbenchmarks (#738)
* Adds GCSFS Microbenchmarks
* Fixing Block size and consistency options in Extended GCSFS Open (#34)
* Separate versioned and non-versioned tests to use different bucket
* Update cleanup logic in tests to empty the bucket instead of deleting the bucket
* Add consistency and blocksize for ext gcsfs
* Merge conflicts resolution
* bucket type paramet for fs test for block size and consistency
* removed unused mocks variables from some tests
* fixing lint errors
* fixed small issue with core tests so it can run with exp flag true
---------
Co-authored-by: Mahalaxmibejugam <60227368+Mahalaxmibejugam@users.noreply.github.com>
* pytest microbenchmarks for seq and random reads both single multi threaded
* multiprocess benchmarks
* script to run tests
* undo settings for bucket names and logs
* benchmark script updates
* file size and bucket type decorators
* file size configuration
* removed zonal config
* Added README
* Readme update
* Moving settings and fixture to tests root
* Readme update
* Readme update
* Ignore benchmark pytests in CI
* benchmark hook fix
* adding skip tests flag
* benchmark plugin conditional enablement
* Fixing PR Comments, simplifying the configuration by doing auto gen
* Fixing PR Comments
* default settings
---------
Co-authored-by: Mahalaxmibejugam <60227368+Mahalaxmibejugam@users.noreply.github.com>
* resolves comments and adds perf micro profiler
* Fixing Block size and consistency options in Extended GCSFS Open (#34)
* Separate versioned and non-versioned tests to use different bucket
* Update cleanup logic in tests to empty the bucket instead of deleting the bucket
* Add consistency and blocksize for ext gcsfs
* Merge conflicts resolution
* bucket type paramet for fs test for block size and consistency
* removed unused mocks variables from some tests
* fixing lint errors
* fixed small issue with core tests so it can run with exp flag true
---------
Co-authored-by: Mahalaxmibejugam <60227368+Mahalaxmibejugam@users.noreply.github.com>
* pytest microbenchmarks for seq and random reads both single multi threaded
* multiprocess benchmarks
* script to run tests
* undo settings for bucket names and logs
* benchmark script updates
* file size and bucket type decorators
* file size configuration
* removed zonal config
* Added README
* Readme update
* Moving settings and fixture to tests root
* Readme update
* Readme update
* Ignore benchmark pytests in CI
* benchmark hook fix
* adding skip tests flag
* benchmark plugin conditional enablement
* Fixing PR Comments, simplifying the configuration by doing auto gen
* Fixing PR Comments
* default settings
* added resource monitoring for benchmarks
* minor refactoring
* moved config to yaml
* lint fixes
* config yaml update for full read suite
* undo zonal file logging changes
* simplify single threaded read
* bringing back requirements
* update readme
* csv generation fix when some tests fail
* psutil install in cloudbuild
* psutil install in cloudbuild
* Removing zonal conditional code and updating config
---------
Co-authored-by: Mahalaxmibejugam <60227368+Mahalaxmibejugam@users.noreply.github.com>
* Parallel File Upload - Microbenchmark Setup (#54)
* Fixing Block size and consistency options in Extended GCSFS Open (#34)
* Separate versioned and non-versioned tests to use different bucket
* Update cleanup logic in tests to empty the bucket instead of deleting the bucket
* Add consistency and blocksize for ext gcsfs
* Merge conflicts resolution
* bucket type paramet for fs test for block size and consistency
* removed unused mocks variables from some tests
* fixing lint errors
* fixed small issue with core tests so it can run with exp flag true
---------
Co-authored-by: Mahalaxmibejugam <60227368+Mahalaxmibejugam@users.noreply.github.com>
* pytest microbenchmarks for seq and random reads both single multi threaded
* multiprocess benchmarks
* script to run tests
* undo settings for bucket names and logs
* benchmark script updates
* file size and bucket type decorators
* file size configuration
* removed zonal config
* Added README
* Readme update
* Moving settings and fixture to tests root
* Readme update
* Readme update
* Ignore benchmark pytests in CI
* benchmark hook fix
* adding skip tests flag
* benchmark plugin conditional enablement
* Fixing PR Comments, simplifying the configuration by doing auto gen
* Fixing PR Comments
* default settings
* added resource monitoring for benchmarks
* minor refactoring
* moved config to yaml
* lint fixes
* config yaml update for full read suite
* undo zonal file logging changes
* simplify single threaded read
* bringing back requirements
* update readme
* csv generation fix when some tests fail
* psutil install in cloudbuild
* psutil install in cloudbuild
* Removing zonal conditional code and updating config
* parallel file creation in setup
* merge conflicts
* merge conflicts - lint fixes
* lint issues
---------
Co-authored-by: Mahalaxmibejugam <60227368+Mahalaxmibejugam@users.noreply.github.com>
* write benchmarks
* Adds mv/rename method implementation for HNS buckets (#727)
* Add mv method for HNS aware implementation
* enable HNS tests in the CI pipeline
* Update super mv call to pass all arguments
* Add more unit tests and make info call optional
* add TODO
* refactor tests and address comments
* address comments
* add debug logs
* update cache logic for rename method
* add TODO
* feat(zb-write): Support write mode in Zonal File (#726)
* Support write mode in ZonalFile and override related methods
* Add unit test for init_aaow method in zb_hns_utils
* Implement zonal write methods
* Remove overwrite parameter since it is not added in AAOW. instead use generation=None to overwrite
* Remove redundant init_aaow method from zonal_file
* update statement for NotImplementedErrors
* set autocommit=false as default for ZonalFile
* add logic to route upload methods to zonal implementation
* Revert "set autocommit=false as default for ZonalFile"
This reverts commit d51c8c3.
* add logic to skip test in test_file itself
* Added comments for clarity
* Added tests for Zonal writes
Close GCSFile before aaow/mrd
* Do not finalize files on close by default
Use simple_flush instead of normal flush
add comments for clarity
* Update requirements to use python-storage 3.7.0
update ci pipeline to run test_zonal_file also
* Update ulimit to 2048 to avoid too many open files error
* Update tests to finalize the object before reading
* Support append mode in ZonalFile
* try removing finalized file if it already exists
* Fix lint error
* fix failing hns rename tests
* Update assertions in mv tests
touch method's side effect shouldn't be counted in the assertion of the rename test
---------
Co-authored-by: Mahalaxmi <mahalaxmib@google.com>
* write test random chunk on every write
* removed zonal conditional code
* Adds mv/rename method implementation for HNS buckets (#727)
* Add mv method for HNS aware implementation
* enable HNS tests in the CI pipeline
* Update super mv call to pass all arguments
* Add more unit tests and make info call optional
* add TODO
* refactor tests and address comments
* address comments
* add debug logs
* update cache logic for rename method
* add TODO
* Listing benchmarking
* Listing benchmarking fixes
* Listing benchmarking scenarios
* refactoring benchmarking code
* Testing after refactoring
* Fixing listing yaml
* refactor file prep
* rename and delete benchmarks
* renaming threads, processes and files fields
* README update and yaml changes
* refactoring read, write setup
* Update read and write config
* fixing zonal write issue and adding data integrity check
* Added unit tests for benchmarks helpers
* Integration test failure fixes
* Integration test failure fixes
---------
Co-authored-by: Mahalaxmibejugam <60227368+Mahalaxmibejugam@users.noreply.github.com>
Co-authored-by: Ankita Luthra <lankita@google.com>
Co-authored-by: suni72 <59387263+suni72@users.noreply.github.com>
Co-authored-by: Mahalaxmi <mahalaxmib@google.com>1 parent 3ed414a commit 9d5b3ff
File tree
32 files changed
+1716
-436
lines changed- cloudbuild
- gcsfs/tests/perf/microbenchmarks
- delete
- listing
- read
- rename
- write
32 files changed
+1716
-436
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
98 | 98 | | |
99 | 99 | | |
100 | 100 | | |
101 | | - | |
| 101 | + | |
102 | 102 | | |
103 | 103 | | |
104 | 104 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
6 | | - | |
7 | | - | |
| 5 | + | |
8 | 6 | | |
9 | | - | |
| 7 | + | |
10 | 8 | | |
11 | | - | |
12 | | - | |
| 9 | + | |
13 | 10 | | |
14 | | - | |
15 | | - | |
16 | | - | |
17 | | - | |
18 | | - | |
19 | | - | |
20 | | - | |
21 | | - | |
22 | | - | |
23 | | - | |
24 | | - | |
25 | | - | |
26 | | - | |
27 | | - | |
28 | | - | |
29 | | - | |
30 | | - | |
31 | | - | |
32 | | - | |
33 | | - | |
34 | | - | |
35 | | - | |
36 | | - | |
37 | | - | |
38 | | - | |
39 | | - | |
40 | | - | |
41 | | - | |
42 | | - | |
| 11 | + | |
43 | 12 | | |
44 | | - | |
45 | | - | |
46 | | - | |
47 | | - | |
48 | | - | |
49 | | - | |
50 | | - | |
51 | | - | |
52 | | - | |
53 | | - | |
54 | | - | |
55 | | - | |
56 | | - | |
57 | | - | |
58 | | - | |
59 | | - | |
60 | | - | |
61 | | - | |
62 | | - | |
| 13 | + | |
63 | 14 | | |
64 | 15 | | |
65 | | - | |
| 16 | + | |
66 | 17 | | |
67 | 18 | | |
68 | | - | |
69 | | - | |
70 | | - | |
| 19 | + | |
71 | 20 | | |
72 | | - | |
| 21 | + | |
73 | 22 | | |
74 | | - | |
75 | | - | |
76 | | - | |
77 | | - | |
| 23 | + | |
78 | 24 | | |
79 | | - | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
80 | 33 | | |
81 | | - | |
82 | | - | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
83 | 37 | | |
84 | | - | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
85 | 41 | | |
86 | | - | |
87 | | - | |
88 | | - | |
89 | | - | |
90 | | - | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
91 | 46 | | |
92 | | - | |
| 47 | + | |
93 | 48 | | |
94 | | - | |
95 | | - | |
96 | | - | |
97 | | - | |
98 | | - | |
99 | | - | |
100 | | - | |
101 | | - | |
102 | | - | |
103 | | - | |
104 | | - | |
| 49 | + | |
105 | 50 | | |
106 | | - | |
| 51 | + | |
| 52 | + | |
107 | 53 | | |
108 | | - | |
109 | | - | |
110 | | - | |
| 54 | + | |
111 | 55 | | |
112 | | - | |
113 | | - | |
114 | | - | |
| 56 | + | |
115 | 57 | | |
116 | | - | |
117 | | - | |
| 58 | + | |
118 | 59 | | |
119 | | - | |
120 | | - | |
121 | | - | |
122 | | - | |
| 60 | + | |
123 | 61 | | |
124 | | - | |
| 62 | + | |
125 | 63 | | |
126 | | - | |
127 | | - | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
128 | 67 | | |
129 | | - | |
| 68 | + | |
130 | 69 | | |
131 | | - | |
| 70 | + | |
132 | 71 | | |
133 | | - | |
134 | | - | |
135 | | - | |
136 | | - | |
137 | | - | |
138 | | - | |
139 | | - | |
| 72 | + | |
140 | 73 | | |
141 | | - | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
142 | 83 | | |
143 | | - | |
| 84 | + | |
144 | 85 | | |
145 | | - | |
| 86 | + | |
146 | 87 | | |
| 88 | + | |
| 89 | + | |
147 | 90 | | |
148 | | - | |
| 91 | + | |
149 | 92 | | |
150 | 93 | | |
151 | | - | |
152 | | - | |
153 | | - | |
154 | | - | |
155 | | - | |
156 | | - | |
157 | | - | |
| 94 | + | |
| 95 | + | |
158 | 96 | | |
159 | | - | |
| 97 | + | |
160 | 98 | | |
161 | 99 | | |
162 | | - | |
163 | | - | |
| 100 | + | |
| 101 | + | |
164 | 102 | | |
165 | | - | |
| 103 | + | |
166 | 104 | | |
167 | 105 | | |
168 | | - | |
169 | | - | |
| 106 | + | |
| 107 | + | |
170 | 108 | | |
171 | | - | |
172 | | - | |
173 | | - | |
174 | | - | |
| 109 | + | |
175 | 110 | | |
176 | 111 | | |
177 | | - | |
178 | | - | |
| 112 | + | |
| 113 | + | |
179 | 114 | | |
180 | | - | |
181 | | - | |
182 | | - | |
183 | | - | |
| 115 | + | |
184 | 116 | | |
185 | 117 | | |
186 | | - | |
187 | | - | |
188 | | - | |
189 | | - | |
190 | | - | |
191 | | - | |
192 | | - | |
193 | | - | |
194 | | - | |
195 | | - | |
196 | | - | |
197 | | - | |
198 | | - | |
199 | | - | |
200 | | - | |
201 | | - | |
202 | | - | |
203 | | - | |
204 | | - | |
205 | | - | |
206 | | - | |
207 | | - | |
208 | | - | |
209 | | - | |
210 | | - | |
211 | | - | |
212 | | - | |
213 | | - | |
214 | | - | |
215 | | - | |
216 | | - | |
| 118 | + | |
217 | 119 | | |
218 | | - | |
| 120 | + | |
219 | 121 | | |
220 | | - | |
221 | | - | |
222 | | - | |
223 | | - | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
0 commit comments