simplify hashable instances, add QColumnName instance, compatibility

uber / queryparser

Parsing and analysis of Vertica, Hive, and Presto SQL.

MIT License

1.08k stars 147 forks source link

simplify hashable instances, add QColumnName instance, compatibility #42

Closed dlthomas-leapyear closed 6 years ago

dlthomas commented 6 years ago

Newer versions of hashable define a Hashable a => Hashable (Identity a) instance, older ones don't. Re-exporting everything while adding that instance if necessary lets us work with both versions.

h4v0kh3l1 commented 6 years ago

If we can put that explanation in the commit message, that looks good to me

dlthomas-leapyear commented 6 years ago

:+1: