perf: increase html-reporter performance #492
Conversation
KuznetsovRoman
force-pushed
the
HERMIONE-1138.perf
branch
3 times, most recently
from
8e4bef6 to
3a7cf13
Compare
| return resolvedUrls; | ||
| } | ||
|
|
||
| async function tryResolveUrl(url) { |
There was a problem hiding this comment.
Рекурсивно проходит по всем вложенным .json базам. Если возникает ошибка при получении данных, возвращаем саму json-базу. Помимо "ничего не сломать", фоллбэк нужен для драфт отчетов
| const results = await Promise.all(urls.map(tryResolveUrl)); | ||
|
|
||
| results.forEach(({jsonUrls, dbUrls}) => { | ||
| resolvedUrls.push(...jsonUrls.concat(dbUrls)); |
There was a problem hiding this comment.
Все urls вставляются в один массив, потому что это позволяет не ломать API слияния отчетов
| @@ -131,7 +131,8 @@ export async function saveStaticFilesToReportDir(htmlReporter: HtmlReporter, plu | |||
| copyToReportDir(destPath, ['report.min.js', 'report.min.css'], staticFolder), | |||
| fs.copy(path.resolve(staticFolder, 'index.html'), path.resolve(destPath, 'index.html')), | |||
| fs.copy(path.resolve(staticFolder, 'icons'), path.resolve(destPath, 'icons')), | |||
| fs.copy(require.resolve('@gemini-testing/sql.js'), path.resolve(destPath, 'sql-wasm.js')), | |||
| @@ -131,7 +131,8 @@ export async function saveStaticFilesToReportDir(htmlReporter: HtmlReporter, plu | |||
| copyToReportDir(destPath, ['report.min.js', 'report.min.css'], staticFolder), | |||
| fs.copy(path.resolve(staticFolder, 'index.html'), path.resolve(destPath, 'index.html')), | |||
| fs.copy(path.resolve(staticFolder, 'icons'), path.resolve(destPath, 'icons')), | |||
| fs.copy(require.resolve('@gemini-testing/sql.js'), path.resolve(destPath, 'sql-wasm.js')), | |||
| fs.copy(require.resolve('@gemini-testing/sql.js/dist/sql-wasm.js'), path.resolve(destPath, 'sql-wasm.js')), | |||
| @@ -131,7 +131,8 @@ export async function saveStaticFilesToReportDir(htmlReporter: HtmlReporter, plu | |||
| copyToReportDir(destPath, ['report.min.js', 'report.min.css'], staticFolder), | |||
| fs.copy(path.resolve(staticFolder, 'index.html'), path.resolve(destPath, 'index.html')), | |||
| fs.copy(path.resolve(staticFolder, 'icons'), path.resolve(destPath, 'icons')), | |||
| fs.copy(require.resolve('@gemini-testing/sql.js'), path.resolve(destPath, 'sql-wasm.js')), | |||
| fs.copy(require.resolve('@gemini-testing/sql.js/dist/sql-wasm.js'), path.resolve(destPath, 'sql-wasm.js')), | |||
| fs.copy(require.resolve('@gemini-testing/sql.js/dist/sql-wasm.wasm'), path.resolve(destPath, 'sql-wasm.wasm')), | |||
|
|
||
| export function groupMeta({group, groupKey, ...restArgs}) { | ||
| export function groupMeta({group, groupKey, diff = group, ...restArgs}) { |
There was a problem hiding this comment.
Здесь и далее diff в функциях - место, куда нужно записывать изменения. По иерархии такие diff равны тому, чему устанавливаются по умолчанию (в данном случае, group)
| group: group.byKey[groupKey], | ||
| result, | ||
| value, | ||
| diff: diff.byKey[groupKey] |
There was a problem hiding this comment.
Так что дальше в функции они прокидываются так же, как и сами их read-only сущности из state
| case actionNames.INIT_GUI_REPORT: | ||
| case actionNames.INIT_STATIC_REPORT: { |
There was a problem hiding this comment.
В этом action, tree не является частью стейта, так что тут можно мутировать его напрямую, что гораздо проще
| @@ -1,4 +1,4 @@ | |||
| import {findLast} from 'lodash'; | |||
There was a problem hiding this comment.
Тут я перенес только action из статического отчета, потому что в gui профит слабый, да и логика там становится намного сложнее, так что оно того не стоит
There was a problem hiding this comment.
Я тут пытался расположить обработку action в том же порядке, но тем не менее, в секции с reduce он отрезолвился плохо. Там можно не смотреть, оставил как было.
Редьюсер распилен на две части:
Первая (статика, без immer.produce):
switch (action.type) {
case actionNames...
....
return applyStateUpdate(state, diff)
Вторая (gui, с immer.produce):
return produce(state, draft => {
switch (action.type) {
case actionNames...
....
break;
})`
| <link rel="preload" href="databaseUrls.json" as="fetch" crossorigin="anonymous"/> | ||
| <link rel="preload" href="sql-wasm.wasm" as="fetch" crossorigin="anonymous"/> |
There was a problem hiding this comment.
Потоков у браузера хватает, чтобы грузить это все вместе с report.min.js параллельно.
KuznetsovRoman
force-pushed
the
HERMIONE-1138.perf
branch
2 times, most recently
from
ebb880d to
c91bcc0
Compare
KuznetsovRoman
force-pushed
the
HERMIONE-1138.perf
branch
from
c91bcc0 to
89b8618
Compare
Пулл-реквест распался: