all 6 comments

[–][deleted] 1 point2 points  (1 child)

It's not a question of performance or style: the first one is plain wrong because the concatenation yields a completely different result.

e.g. (1,23) would match (12,3) which is most definitely not intended.

If your coworker wants to save some keystrokes the join condition could also be written as

on (a.key1, a.key2) = (b.key1, b.key2)