smarr / ReBenchDB

ReBenchDB records benchmark results and provides customizable reporting to track and analyze run-time performance of software programs.
MIT License
12 stars 6 forks source link

Use JOIN ... USING instead of JOIN ... ON #185

Closed smarr closed 6 months ago

smarr commented 6 months ago

This is the start of a PR to move from using JOIN ... ON to the JOIN ... USING, which only requires to name the columns on which to join.

Though, while I do like the idea of getting rid of columns just named id, I don't like that the JOIN clauses don't make it explicit which tables are actually joined. While this makes things somewhat redundant, it also allows one to read an SQL query in isolation and see which tables are linked together.

So, this PR is mostly for archival. I'll close it without merging. (It's also still incomplete).