geldata / geldata/gel-cli

Unnecessary `conv_expr` for changing cardinality on computed

Open
#1,124 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Rust
Stars
175
Forks
32
PR merge metrics
No merged PRs in 30d

Description

- EdgeDB Version: 3.3
- EdgeDB CLI Version: 3.5.0+e1ad387
- OS Version: macos 13.5

Schema
```
type Target_ {
multi link latestSource := (
select . y
Please specify an expression in order to convert link 'latestSource' of object type 'default::Target_' to 'single' cardinality:
conv_expr> (SELECT .latestSource LIMIT 1)
```
The expression seems unnecessary since `latestSource` is a computed. This one is obviously easy, and in fact the CLI fills it in for you (!!) but filing in case this would affect other migration changes to computeds or is a hint of a deeper problem.

Contributor guide

No contributing guide indexed for this repository

Research direction

Reproduce the migration in the issue using the Target_ and Source_ schema, then trace the CLI migration prompt for changing a computed link from multi to single cardinality. Check how computed links are handled before requesting conv_expr. Done means this change no longer asks for an unnecessary conversion expression while other cardinality conversions still behave correctly.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
cli, databases
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.