finos / legend-studio

Legend Studio
https://legend.finos.org
Apache License 2.0
87 stars 114 forks source link

datacube: application infrastructure improvements #3350

Closed akphi closed 1 month ago

changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: 54cfc1994fc8de251562f971ee754923ff6ddd77

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

This PR includes changesets to release 36 packages | Name | Type | | --------------------------------------------- | ----- | | @finos/legend-application-repl | Patch | | @finos/legend-application-pure-ide-deployment | Patch | | @finos/legend-extension-dsl-data-space-studio | Patch | | @finos/legend-application-studio-deployment | Patch | | @finos/legend-extension-store-service-store | Patch | | @finos/legend-vscode-extension-dependencies | Patch | | @finos/legend-application-query-deployment | Patch | | @finos/legend-application-studio-bootstrap | Patch | | @finos/legend-application-query-bootstrap | Patch | | @finos/legend-application-repl-deployment | Patch | | @finos/legend-extension-dsl-data-quality | Patch | | @finos/legend-extension-store-relational | Patch | | @finos/legend-server-showcase-deployment | Patch | | @finos/legend-extension-dsl-persistence | Patch | | @finos/legend-extension-store-flat-data | Patch | | @finos/legend-extension-dsl-data-space | Patch | | @finos/legend-extension-dsl-diagram | Patch | | @finos/legend-extension-dsl-service | Patch | | @finos/legend-application-pure-ide | Patch | | @finos/legend-extension-assortment | Patch | | @finos/legend-application-studio | Patch | | @finos/legend-extension-dsl-text | Patch | | @finos/legend-application-query | Patch | | @finos/legend-server-showcase | Patch | | @finos/legend-query-builder | Patch | | @finos/legend-application | Patch | | @finos/legend-server-sdlc | Patch | | @finos/legend-dev-utils | Patch | | @finos/eslint-plugin-legend-studio | Patch | | @finos/legend-shared | Patch | | @finos/legend-graph | Patch | | @finos/legend-lego | Patch | | @finos/legend-art | Patch | | @finos/legend-extension-tracer-zipkin | Patch | | @finos/legend-server-depot | Patch | | @finos/legend-storage | 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

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 1.57959% with 1620 lines in your changes missing coverage. Please review.

Project coverage is 46.04%. Comparing base (11e0740) to head (54cfc19).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #3350 +/- ## ========================================== - Coverage 46.13% 46.04% -0.09% ========================================== Files 2089 2089 Lines 358306 358279 -27 Branches 15299 10822 -4477 ========================================== - Hits 165298 164970 -328 - Misses 192282 192677 +395 + Partials 726 632 -94 ``` | [Files](https://app.codecov.io/gh/finos/legend-studio/pull/3350?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos) | Coverage Δ | | |---|---|---| | [...pl/src/stores/dataCube/core/DataCubeQueryEngine.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3350?src=pr&el=tree&filepath=packages%2Flegend-application-repl%2Fsrc%2Fstores%2FdataCube%2Fcore%2FDataCubeQueryEngine.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWFwcGxpY2F0aW9uLXJlcGwvc3JjL3N0b3Jlcy9kYXRhQ3ViZS9jb3JlL0RhdGFDdWJlUXVlcnlFbmdpbmUudHM=) | `94.38% <100.00%> (-0.03%)` | :arrow_down: | | [...ores/dataCube/core/DataCubeQuerySnapshotBuilder.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3350?src=pr&el=tree&filepath=packages%2Flegend-application-repl%2Fsrc%2Fstores%2FdataCube%2Fcore%2FDataCubeQuerySnapshotBuilder.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWFwcGxpY2F0aW9uLXJlcGwvc3JjL3N0b3Jlcy9kYXRhQ3ViZS9jb3JlL0RhdGFDdWJlUXVlcnlTbmFwc2hvdEJ1aWxkZXIudHM=) | `97.39% <100.00%> (ø)` | | | [.../dataCube/grid/DataCubeGridQuerySnapshotBuilder.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3350?src=pr&el=tree&filepath=packages%2Flegend-application-repl%2Fsrc%2Fstores%2FdataCube%2Fgrid%2FDataCubeGridQuerySnapshotBuilder.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWFwcGxpY2F0aW9uLXJlcGwvc3JjL3N0b3Jlcy9kYXRhQ3ViZS9ncmlkL0RhdGFDdWJlR3JpZFF1ZXJ5U25hcHNob3RCdWlsZGVyLnRz) | `0.00% <ø> (ø)` | | | [...-group/mapping-editor/EnumerationMappingEditor.tsx](https://app.codecov.io/gh/finos/legend-studio/pull/3350?src=pr&el=tree&filepath=packages%2Flegend-application-studio%2Fsrc%2Fcomponents%2Feditor%2Feditor-group%2Fmapping-editor%2FEnumerationMappingEditor.tsx&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWFwcGxpY2F0aW9uLXN0dWRpby9zcmMvY29tcG9uZW50cy9lZGl0b3IvZWRpdG9yLWdyb3VwL21hcHBpbmctZWRpdG9yL0VudW1lcmF0aW9uTWFwcGluZ0VkaXRvci50c3g=) | `85.15% <100.00%> (ø)` | | | [...oup/service-editor/testable/ServiceTestsEditor.tsx](https://app.codecov.io/gh/finos/legend-studio/pull/3350?src=pr&el=tree&filepath=packages%2Flegend-application-studio%2Fsrc%2Fcomponents%2Feditor%2Feditor-group%2Fservice-editor%2Ftestable%2FServiceTestsEditor.tsx&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWFwcGxpY2F0aW9uLXN0dWRpby9zcmMvY29tcG9uZW50cy9lZGl0b3IvZWRpdG9yLWdyb3VwL3NlcnZpY2UtZWRpdG9yL3Rlc3RhYmxlL1NlcnZpY2VUZXN0c0VkaXRvci50c3g=) | `69.09% <100.00%> (ø)` | | | [...tion-studio/src/stores/editor/ExplorerTreeState.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3350?src=pr&el=tree&filepath=packages%2Flegend-application-studio%2Fsrc%2Fstores%2Feditor%2FExplorerTreeState.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWFwcGxpY2F0aW9uLXN0dWRpby9zcmMvc3RvcmVzL2VkaXRvci9FeHBsb3JlclRyZWVTdGF0ZS50cw==) | `61.98% <100.00%> (-2.74%)` | :arrow_down: | | [...ckageableElements/mapping/V1\_EnumerationMapping.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3350?src=pr&el=tree&filepath=packages%2Flegend-graph%2Fsrc%2Fgraph-manager%2Fprotocol%2Fpure%2Fv1%2Fmodel%2FpackageableElements%2Fmapping%2FV1_EnumerationMapping.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWdyYXBoL3NyYy9ncmFwaC1tYW5hZ2VyL3Byb3RvY29sL3B1cmUvdjEvbW9kZWwvcGFja2FnZWFibGVFbGVtZW50cy9tYXBwaW5nL1YxX0VudW1lcmF0aW9uTWFwcGluZy50cw==) | `100.00% <100.00%> (ø)` | | | [...formation/pureGraph/from/V1\_DatabaseTransformer.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3350?src=pr&el=tree&filepath=packages%2Flegend-graph%2Fsrc%2Fgraph-manager%2Fprotocol%2Fpure%2Fv1%2Ftransformation%2FpureGraph%2Ffrom%2FV1_DatabaseTransformer.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWdyYXBoL3NyYy9ncmFwaC1tYW5hZ2VyL3Byb3RvY29sL3B1cmUvdjEvdHJhbnNmb3JtYXRpb24vcHVyZUdyYXBoL2Zyb20vVjFfRGF0YWJhc2VUcmFuc2Zvcm1lci50cw==) | `81.14% <100.00%> (ø)` | | | [...raph/from/V1\_GenerationSpecificationTransformer.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3350?src=pr&el=tree&filepath=packages%2Flegend-graph%2Fsrc%2Fgraph-manager%2Fprotocol%2Fpure%2Fv1%2Ftransformation%2FpureGraph%2Ffrom%2FV1_GenerationSpecificationTransformer.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWdyYXBoL3NyYy9ncmFwaC1tYW5hZ2VyL3Byb3RvY29sL3B1cmUvdjEvdHJhbnNmb3JtYXRpb24vcHVyZUdyYXBoL2Zyb20vVjFfR2VuZXJhdGlvblNwZWNpZmljYXRpb25UcmFuc2Zvcm1lci50cw==) | `92.39% <100.00%> (ø)` | | | [...Graph/to/helpers/V1\_FileGenerationBuilderHelper.ts](https://app.codecov.io/gh/finos/legend-studio/pull/3350?src=pr&el=tree&filepath=packages%2Flegend-graph%2Fsrc%2Fgraph-manager%2Fprotocol%2Fpure%2Fv1%2Ftransformation%2FpureGraph%2Fto%2Fhelpers%2FV1_FileGenerationBuilderHelper.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos#diff-cGFja2FnZXMvbGVnZW5kLWdyYXBoL3NyYy9ncmFwaC1tYW5hZ2VyL3Byb3RvY29sL3B1cmUvdjEvdHJhbnNmb3JtYXRpb24vcHVyZUdyYXBoL3RvL2hlbHBlcnMvVjFfRmlsZUdlbmVyYXRpb25CdWlsZGVySGVscGVyLnRz) | `100.00% <100.00%> (+4.44%)` | :arrow_up: | | ... and [68 more](https://app.codecov.io/gh/finos/legend-studio/pull/3350?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos) | | ... and [333 files with indirect coverage changes](https://app.codecov.io/gh/finos/legend-studio/pull/3350/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=finos)
MauricioUyaguari commented 1 month ago

@akphi as mentioned, lets split bump dependencies into its own PR so the team is aware of what has changed. makes it hard if it is buried under 146 changes of repl. those changes affect the rest of the application and not just repl.