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.

Sign up for GitHub

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

Jump to bottom

Add Int64 type #330

Closed
tjungblu opened this issue Nov 15, 2023 · 0 comments · Fixed by #332
Closed

Add Int64 type #330

tjungblu opened this issue Nov 15, 2023 · 0 comments · Fixed by #332

Comments

@tjungblu
Copy link
Contributor

Hey there,

I wanted to spend some time to analyse byte sizes in etcd databases with an octosql plugin I wrote.

Unfortunately, there's only 32 bit Int, which easily overflows. Would you be interested to have a 64 bit integer as a type for octosql?
tjungblu added a commit to tjungblu/octosql that referenced this issue Apr 23, 2024
@tjungblu
Add int64 support
e667752 
fixes cube2222#330

Signed-off-by: Thomas Jungblut <tjungblu@redhat.com>
@tjungblu tjungblu mentioned this issue Apr 23, 2024
Closed
tjungblu added a commit to tjungblu/octosql that referenced this issue Apr 23, 2024
@tjungblu
Add int64 support
9003cbb 
fixes cube2222#330

Signed-off-by: Thomas Jungblut <tjungblu@redhat.com>
tjungblu added a commit to tjungblu/octosql that referenced this issue Apr 24, 2024
@tjungblu
mark int type explicitly as int64
3c2339c 
fixes cube2222#330

Signed-off-by: Thomas Jungblut <tjungblu@redhat.com>
@tjungblu tjungblu mentioned this issue Apr 24, 2024
Merged
cube2222 pushed a commit that referenced this issue May 11, 2024
@tjungblu
mark int type explicitly as int64 (#332)
4a66fe0 
* mark int type explicitly as int64

fixes #330

Signed-off-by: Thomas Jungblut <tjungblu@redhat.com>

* use 32bit truncation for parquet

---------

Signed-off-by: Thomas Jungblut <tjungblu@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

mark int type explicitly as int64 tjungblu/octosql
Add int64 support tjungblu/octosql
1 participant
@tjungblu