Skip to content

DEV-1712 Stage 8: order-only hidden slots + plan-time validations (order-only refs, CMA trim, partition_by) - #274

Merged
ZmeiGorynych merged 8 commits into
egor/dev-1703-comprehensive-approach-expressions-crossing-joins-on-thefrom
egor/dev-1712-dev-1703-stage-8-hidden-slots-plan-time-validations-order
Aug 4, 2026
Merged

DEV-1712 Stage 8: order-only hidden slots + plan-time validations (order-only refs, CMA trim, partition_by)#274
ZmeiGorynych merged 8 commits into
egor/dev-1703-comprehensive-approach-expressions-crossing-joins-on-thefrom
egor/dev-1712-dev-1703-stage-8-hidden-slots-plan-time-validations-order

DEV-1712 Stage 8: order-only hidden slots + plan-time validations

f5e588b
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Aug 3, 2026 in 1m 25s

Annotations

Check warning on line 647 in tests/test_dev1712_order_only_hidden_slots.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions.

See more on https://sonarcloud.io/project/issues?id=MotleyAI_slayer&issues=AZ_IiRD-7ZloDB7cAhbi&open=AZ_IiRD-7ZloDB7cAhbi&pullRequest=274

Check failure on line 405 in slayer/engine/planning.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 26 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=MotleyAI_slayer&issues=AZ_IiRFf7ZloDB7cAhbk&open=AZ_IiRFf7ZloDB7cAhbk&pullRequest=274

Check warning on line 1362 in tests/test_nested_dag_cross_stage_refs.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions.

See more on https://sonarcloud.io/project/issues?id=MotleyAI_slayer&issues=AZ_IiQ-27ZloDB7cAhbf&open=AZ_IiQ-27ZloDB7cAhbf&pullRequest=274

Check warning on line 246 in tests/test_dev1712_order_only_hidden_slots.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions.

See more on https://sonarcloud.io/project/issues?id=MotleyAI_slayer&issues=AZ_IiRD-7ZloDB7cAhbh&open=AZ_IiRD-7ZloDB7cAhbh&pullRequest=274

Check warning on line 622 in slayer/engine/stage_planner.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move this assignment out of the argument list; ":=" operator is confusing in this context.

See more on https://sonarcloud.io/project/issues?id=MotleyAI_slayer&issues=AZ_IiRFx7ZloDB7cAhbl&open=AZ_IiRFx7ZloDB7cAhbl&pullRequest=274

Check warning on line 225 in tests/test_dev1712_order_only_hidden_slots.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions.

See more on https://sonarcloud.io/project/issues?id=MotleyAI_slayer&issues=AZ_IiRD-7ZloDB7cAhbg&open=AZ_IiRD-7ZloDB7cAhbg&pullRequest=274

Check warning on line 696 in tests/test_dev1712_order_only_hidden_slots.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This assertion is too broad; use a more specific exception type or check the exception message.

See more on https://sonarcloud.io/project/issues?id=MotleyAI_slayer&issues=AZ_IiRD-7ZloDB7cAhbj&open=AZ_IiRD-7ZloDB7cAhbj&pullRequest=274