Handling of numeric columns.

[ivanceras]

I think numeric columns should be handled in rust-postgres. Numeric columns are very common data types for high precision requirements.
Looking at diesel-pg implementation, it seems to be converting the raw vec to bigdecimal and bigint.

https://github.com/diesel-rs/diesel/blob/master/diesel/src/pg/types/numeric.rs

bigdecimal
num_bigint

I couldn't wrap my head around the implementation, it's more complex that I though it would be.

[jwilm]

I would also enjoy having this feature. In the mean time, I've been querying them as colname::text and parsing the result.

[sfackler]

Sure, I'd be happy to take a PR adding support via bigdecimal!

[sfackler]

How is NaN handled?

[ivanceras]

Shouldn't be Nan just be treated as Error in Result<T,Error> ?

[sfackler]

It could be, but it seems like you'd want to be able to at least have the ability to read a NaN out of the database. I guess it could just be a wrapper enum though.

[ivanceras]

@sfackler yes, the Nan types is a variant in the enum.

https://github.com/diesel-rs/diesel/blob/master/diesel/src/pg/types/floats/mod.rs#L12

#[derive(Debug, Clone, PartialEq, Eq)]
pub enum PgNumeric {
    Positive {
        weight: i16,
        scale: u16,
        digits: Vec<i16>,
    },
    Negative {
        weight: i16,
        scale: u16,
        digits: Vec<i16>,
    },
    NaN,
}

I may have to play a bit more on these numeric types before I could make a PR.

[ivanceras]

I already have a working numeric handling for postgres.
https://github.com/ivanceras/rustorm/blob/master/src/pg/numeric.rs
I'm still figuring out how to integrate this into the rust-postgres source code.

[fromheten]

This is a features I'd like to have in rust-postgres.

In order to put my money where my mouth is I therefore want to set a €100 bounty on this.

I'll pay the money to whoever submits a patch (under the same licence as this project, MIT) that lets me do #[derive(FromSql, ToSql)] on a struct where one member is of type num_bigint::BigUint or equivalent (I'm just looking for positive integers in my application). If it uses a different but equivalent library than num_bigint that's all OK for me. What I'm looking for is being able to have an any-precision positive integer in my struct, and be able to treat it as a number type in PG, and for this to work as great as the rest of this library (great work @sfackler).

Whoever does that and sends me an invoice for €100 will get it! Can pay in Bitcoin Core/Bitcoin Cash, Ethereum, SEPA transaction or cash plus a beer or coffee if you're around Berlin, Germany.

I'll give it a go myself as well, but hope someone else will be faster than me. @ivanceras what you have seems like a great start.

[ivanceras]

I was trying to integrate numeric into rust-postgres but I couldn't get the current tests pass.
https://hub.docker.com/r/sfackler/rust-postgres-test/ looks like the test image is private. @sfackler can you opensource the Dockerfile used to build the image for the rust-postgres-test setup?

[kestred]

I'd particularly like to vote on Numeric support for bigdecimal, with the ongoing optimisim that eventually bigdecimal will at some point become part of num again.

I'm currently using the same work around as @jwilm (mynumeric::text); but if a feature for the bigdecimal crate would be welcome, I'd be happy to put together a PR.

[sfackler]

@ivanceras Hmm, it seems to be public as far as I can tell - I don't think Circle would be able to pull it otherwise.

Definitely happy to support bigdecimal/whatever else!

[sfackler]

The docker build for the image is over here though in any case: https://github.com/sfackler/rust-postgres/tree/master/docker

[kdar]

rust_decimal has an implementation for rust-postgres it it helps anyone: https://github.com/paupino/rust-decimal/blob/126271f9ef9a1e599bcd55eee70657486be0530f/src/postgres.rs

rust_decimal performs pretty well except for dividing, in case anyone was curious:

test tests::bench_bigdecimal_add   ... bench:     541,332 ns/iter (+/- 124,543)
test tests::bench_bigdecimal_div   ... bench:     559,528 ns/iter (+/- 190,289)
test tests::bench_bigdecimal_mul   ... bench:     493,652 ns/iter (+/- 16,276)
test tests::bench_bigdecimal_sub   ... bench:     344,488 ns/iter (+/- 58,812)
test tests::bench_decimal_add      ... bench:     112,622 ns/iter (+/- 5,842)
test tests::bench_decimal_div      ... bench:     362,076 ns/iter (+/- 75,458)
test tests::bench_decimal_mul      ... bench:     239,117 ns/iter (+/- 31,331)
test tests::bench_decimal_sub      ... bench:     542,171 ns/iter (+/- 34,186)
test tests::bench_rust_decimal_add ... bench:     104,577 ns/iter (+/- 4,711)
test tests::bench_rust_decimal_div ... bench:   1,201,631 ns/iter (+/- 77,024)
test tests::bench_rust_decimal_mul ... bench:      55,916 ns/iter (+/- 14,682)
test tests::bench_rust_decimal_sub ... bench:      73,354 ns/iter (+/- 3,539)

[fromheten]

@kdar that seems to be solving the problem I had at hand. Thanks for sharing! Gonna give it a go on Monday.

[jspeis]

Would anyone happen to know how I could integrate the rust_decimal support to work with the master branch version of tokio-postgres?