evidence-dev / evidence

Business intelligence as code: build fast, interactive data visualizations in SQL and markdown
https://evidence.dev
MIT License
4.5k stars 216 forks source link

Add parquet building logs + profiling to `evidence *` commands #2654

Closed csjh closed 1 month ago

csjh commented 1 month ago

Description

Adds logging to build-parquet.js (mainly to diagnose slow builds), also generically adds a --profile flag to all evidence * commands which profiles the duration of the command's execution

Checklist

changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: a2e428e5f32c1da5b9101a58ee9a6504692bcbab

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 12 packages | Name | Type | | --------------------------------- | ----- | | @evidence-dev/sdk | Patch | | @evidence-dev/universal-sql | Patch | | my-evidence-project | Patch | | @evidence-dev/evidence | Patch | | @evidence-dev/component-utilities | Patch | | @evidence-dev/core-components | Patch | | @evidence-dev/tailwind | Patch | | @evidence-dev/components | Patch | | evidence-test-environment | Patch | | @evidence-dev/plugin-connector | Patch | | e2e-spa | Patch | | e2e-themes | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 17, 2024 7:57pm
next-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 17, 2024 7:57pm
netlify[bot] commented 1 month ago

Deploy Preview for next-docs-evidence ready!

Name Link
Latest commit a2e428e5f32c1da5b9101a58ee9a6504692bcbab
Latest deploy log https://app.netlify.com/sites/next-docs-evidence/deploys/671169799919ca0007bc2edc
Deploy Preview https://deploy-preview-2654--next-docs-evidence.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] commented 1 month ago

Deploy Preview for evidence-development-workspace ready!

Name Link
Latest commit a2e428e5f32c1da5b9101a58ee9a6504692bcbab
Latest deploy log https://app.netlify.com/sites/evidence-development-workspace/deploys/671169794dd34400096434f1
Deploy Preview https://deploy-preview-2654--evidence-development-workspace.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] commented 1 month ago

Deploy Preview for evidence-test-env ready!

Name Link
Latest commit a2e428e5f32c1da5b9101a58ee9a6504692bcbab
Latest deploy log https://app.netlify.com/sites/evidence-test-env/deploys/67116979199ac400085f0809
Deploy Preview https://deploy-preview-2654--evidence-test-env.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

csjh commented 1 month ago

@ItsMeBrianD any thoughts on the log.measure usage? I'm not sure if I'm a massive fan of the

let done;
if (isDebug()) { ... }

pattern

Could keep track of active labels internally and add a log.measureEnd ala console.timeEnd?