geldata / geldata/gel-rust

Combine RawCodec and Queryable?

Open
#40 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
231
Forks
30
PR merge metrics
No merged PRs in 30d

Description

What is the benefit of having `RawCodec` in addition to `Queryable`? It has a subset of the functionality, but for every type you'd want to implement `RawCodec` for, you'd also want to implement `Queryable` for. (It's also never used as trait, so if there are exceptions, you could just use a free function as helper)

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the RawCodec and Queryable trait definitions and searching their implementations and usages across the crate. Compare their responsibilities and usage patterns, then document a decided consolidation or retention approach and update affected APIs and tests accordingly.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend-api-design
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.