search

just-bi / odxl

Open Data Export Layer
Apache License 2.0
13 stars 7 forks source link

Add BigInt aka. long aka. Int64 Support for the odatafilterparser and… #30

Closed mathias-mcb closed 7 years ago

mathias-mcb commented 7 years ago

… query builder

If you work with column types "BigInt" your OData endpoint requires filter expressions like: $filter=(CD lt 1487257323585L)

So my column is CD and my filter value is 1487257323585.

In my HANA OData implementation the endpoint wouldn't accept $filter=(CD lt 1487257323585) because the value exceeds Integer range.

So I think odxl should support the 123L notation.

rpbouman commented 7 years ago

Thanks! Much appreciated :)

After accepting the pull request I made a change in the querybuilder: instead of immediately returning the value, I added it to the str variable to maintain a single point of exit.