Skip to content

Conversation

@mcheshkov
Copy link
Contributor

Check List

  • Tests have been run in packages where changes made if available
  • Linter has been run for changed code
  • Tests for the changes have been added if not covered yet
  • Docs have been added / updated if required

Description of Changes Made (if issue reference is not provided)

Without this when measure was a member expression m.measure.name can be undefined, and different measures will have same key in collectRootMeasureToHieararchy

Copy link
Member

@KSDaemon KSDaemon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻

@KSDaemon KSDaemon marked this pull request as ready for review June 6, 2025 15:35
@KSDaemon KSDaemon requested a review from a team as a code owner June 6, 2025 15:35
@KSDaemon KSDaemon changed the title [WIP] fix(schema-compiler): Use member expression definition as measure key fix(schema-compiler): Use member expression definition as measure key Jun 6, 2025
@KSDaemon KSDaemon self-assigned this Jun 13, 2025
…n as measure key

Without this when measure was a member expression m.measure.name can be undefined, and different measures will have same key in collectRootMeasureToHieararchy
@KSDaemon KSDaemon force-pushed the member-expression-measure-key branch from d958e57 to 547c394 Compare June 25, 2025 17:07
@KSDaemon KSDaemon merged commit e478d0e into master Jun 26, 2025
84 of 85 checks passed
@KSDaemon KSDaemon deleted the member-expression-measure-key branch June 26, 2025 08:42
Frank-TXS pushed a commit to Helge-TXS/cube that referenced this pull request Aug 5, 2025
…cube-js#9154)

Without this when measure was a member expression m.measure.name can be undefined, and different measures will have same key in collectRootMeasureToHieararchy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants