edgedb-rust icon indicating copy to clipboard operation
edgedb-rust copied to clipboard

Combine RawCodec and Queryable?

Open CodesInChaos opened this issue 5 years ago • 1 comments

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)

CodesInChaos avatar Aug 17 '20 12:08 CodesInChaos

Well, yes, it looks like ugly design artifact. I have mixed feeling, though: Queryable is meant to mean a high level thing, i.e. something that you can query from the database, so using it from the inside of Codec sounds a bit weird.

tailhook avatar Aug 17 '20 13:08 tailhook