Initial stab at this, still has some failures but you get the idea.
Closes #3421164
Merge request reports
Activity
added 1 commit
- 449ce398 - Undo a fix for an older version of opentelemetry
added 1 commit
- f906acbc - Added second anonymous block, slight update to query comparisons
added 1 commit
- 9a07e3bd - Add final block before we start hunting the discrepancies
added 38 commits
-
a665bba7...2d17900f - 19 commits from branch
project:11.x
- 2d17900f...a427feb5 - 9 earlier commits
- ea0af433 - Assert the right thing.
- 83589a34 - remove s
- e62ad911 - Increase upper range to 146.
- 1e1a1886 - Increase range again.
- 595250e5 - Tag
- 9762d00e - Add a range to the assertion.
- 99e373d3 - Add a range to the assertion.
- 4e288424 - Add range.
- a11769c1 - Add range to assertion.
- 160473a1 - Merge branch '3421881-cache-tags' into 3421164-log-every-individual
Toggle commit list-
a665bba7...2d17900f - 19 commits from branch
added 1 commit
- 76cf9789 - Remove all query aliasing in favor of making the queries invariable
added 7 commits
- ea0fea87 - Apply 1 suggestion(s) to 1 file(s)
- 53585cac - Apply 1 suggestion(s) to 1 file(s)
- f0344f4a - Apply 1 suggestion(s) to 1 file(s)
- 575e1849 - Switch to enum, also make sure we send the cache tag data to OpenTelemetry
- 78c7fc67 - codesniffer fix
- f9b9a5be - Fix nits
- a4f54a3a - Merge branch '3421881-cache-tags' into 3421164-log-every-individual
Toggle commit listadded 1 commit
- 3537fcc1 - Separate out all the query massaging so it is easier to read and can be overridden or extended.
added 45 commits
-
3537fcc1...d66403c9 - 44 commits from branch
project:11.x
- 2f8f7933 - Merge remote-tracking branch 'origin/11.x' into 3421164-log-every-individual
-
3537fcc1...d66403c9 - 44 commits from branch
added 3 commits
-
2f8f7933...213906e2 - 2 commits from branch
project:11.x
- 69ab40f9 - Merge remote-tracking branch 'origin/11.x' into 3421164-log-every-individual
-
2f8f7933...213906e2 - 2 commits from branch
- Resolved by Kristiaan Van den Eynde
- Resolved by catch
253 254 // Inline query arguments and log the query. 255 $query = str_replace(array_keys($args), array_values($this->quoteQueryArgs($args)), $query); 256 $performance_data->logQuery($query); 257 } 258 259 /** 260 * Wraps query arguments in double quotes if they're a string. 261 * 262 * @param array $args 263 * The raw query arguments. 264 * 265 * @return array 266 * The conditionally quoted query arguments. 267 */ 268 protected function quoteQueryArgs(array $args): array { changed this line in version 14 of the diff
168 176 return $performance_data; 169 177 } 170 178 179 /** 180 * Logs a query in the performance data. 181 * 182 * @param \Drupal\Tests\PerformanceData $performance_data 183 * The performance data object to log the query on. 184 * @param string $query 185 * The raw query. 186 * @param array $args 187 * The query arguments. 188 */ 189 protected function logQuery(PerformanceData $performance_data, string $query, array $args): void { changed this line in version 14 of the diff
added 1 commit
- f3307f3e - Get rid of query aliasing altogether and find a clever way to placeholder the...
added 3 commits
-
f3307f3e...3923a9f8 - 2 commits from branch
project:11.x
- 1e7ccafe - Merge remote-tracking branch 'origin/11.x' into 3421164-log-every-individual
-
f3307f3e...3923a9f8 - 2 commits from branch
Please register or sign in to reply