Remove use of Row#getAnyOption from FinaglePostgresDecoders

[jonathan-ostrander]

Fixes #1844

Problem

The decoders that are not direct in FinaglePostgresDecoders rely on Row#getAnyOption. Row#getAnyOption relies on custom receiving functions being defined on the PostgresClient. postgres-finagle could define these by default (which it probably should), but by default it does not. When custom receiving functions are not defined, Row#getAnyOption always returns None.

Solution

This change introduces an orElse method on FinaglePostgresDecoder which allows for composition of ValueDecoders that was previous being accomplished by PartialFunctions. This solution removes the need for ClassTag (although we may want to keep it to keep error messages consistent) as well as the reliance on Row#getAnyOption.

Notes

This removes the definition of decoder[T] which I do not see much value in since it requires custom receive functions to be defined for T in order to work.

Checklist

• Unit test all changes
• Update README.md if applicable
• Add [WIP] to the pull request title if it's work in progress
• Squash commits that aren't meaningful changes
• Run sbt scalariformFormat test:scalariformFormat to make sure that the source files are formatted

@getquill/maintainers

[juliano]

Thank you for that! 💯