graphile / graphile/crystal

Have PgSelectStep leverage unary steps

Open
#1,974 1 comment 0 reactions 0 assignees View on GitHub
✨ feature
Dominant language
TypeScript
Stars
12.9k
Forks
625
Avg merge
5h 23m
Merged PRs (30d)
24

Description

Relates to #1973; PgSelectStep and related classes should be able to `addUnaryDependency` for first/etc and thus not need to do such complex SQL for first/last/cursor/etc. Should be able to use this to eliminate some `eval` calls also?

Contributor guide

Open the contributing guide

Research direction

Start with PgSelectStep and the related classes, then read the discussion in #1973. Trace how first/last/cursor operations currently build SQL and where eval calls are used. Done means unary dependencies can handle these operations and the unnecessary SQL complexity and eval calls are removed.

Written by the indexing model from the issue text.

Assessment

Tech stack
postgresql, typescript
Domain
api, backend, databases
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.