Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

int8 arrays parsed as numbers, float arrays parsed as strings #452

Closed
strk opened this issue Oct 11, 2013 · 4 comments
Closed

int8 arrays parsed as numbers, float arrays parsed as strings #452

strk opened this issue Oct 11, 2013 · 4 comments
Labels

Comments

@strk
Copy link
Contributor

strk commented Oct 11, 2013

I was testing the recent numeric parsing and found out that:

  1. int8 is parsed as string, but _int8 is parsed as an array of numbers (should be strings)
  2. int4 is parsed as number, but _int4 is parsed as an array of strings (should be numbers)
  3. float8 is parsed as number, but _float8 is parsed as an array of strings (should be numbers)

This is the default as of version 2.6.2

@brianc
Copy link
Owner

brianc commented Oct 14, 2013

Ah, yeah that's yucky. I didn't modify the array parsing when I switched things back over to the more pedantic type parsing.

@strk
Copy link
Contributor Author

strk commented Nov 19, 2013

See also #378

brianc added a commit that referenced this issue Feb 26, 2014
improve support for int arrays and float arrays
@benesch
Copy link
Contributor

benesch commented May 8, 2014

@brianc seems like this was resolved by #501?

(Just eyeballing the diff; haven't actually verified.)

@brianc
Copy link
Owner

brianc commented May 8, 2014

Yeah you're right! thanks. :)

@brianc brianc closed this as completed May 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants