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

parquet bug with both uint and int segarray in dataframe #3788

Closed
stress-tess opened this issue Sep 24, 2024 · 0 comments · Fixed by #3790
Closed

parquet bug with both uint and int segarray in dataframe #3788

stress-tess opened this issue Sep 24, 2024 · 0 comments · Fixed by #3790
Assignees
@stress-tess
Labels
bug Something isn't working File IO Arkouda file IO capabilities

Comments

@stress-tess
Copy link
Member

I found the cause of #3784, but the test case was still failing. It turned up this bug when we have both uint and int segarray in a dataframe

note: slice index was 5..9 but array bounds are 0..4

arkouda/src/ParquetMsg.chpl

Line 1612 in 8dae0c5

var int_vals: [0..#locSize_int] int; //int and uint written the same so no conversions will be needed

@stress-tess stress-tess added bug Something isn't working File IO Arkouda file IO capabilities labels Sep 24, 2024
@stress-tess stress-tess self-assigned this Sep 24, 2024
@stress-tess stress-tess changed the title separate uint and int vals in parquet message parquet bug with both uint and int segarray in dataframe Sep 24, 2024
stress-tess added a commit to stress-tess/arkouda that referenced this issue Sep 24, 2024
@stress-tess
Fixes Bears-R-Us#3783, Bears-R-Us#3784, Bears-R-Us#3788: multilocale …
da547f6 
…io test failures

This PR fixes multiple bugs found when running `io_tests` with more than two locales in `ARKOUDA_RUNNING_MODE=client`

fixes Bears-R-Us#3783, fixes Bears-R-Us#3784, fixes Bears-R-Us#3788
@stress-tess stress-tess mentioned this issue Sep 25, 2024
Merged
github-merge-queue bot pushed a commit that referenced this issue Sep 27, 2024
@stress-tess
Fixes #3783, #3784, #3788: multilocale io test failures (#3790)
3cc2a5b 
This PR fixes multiple bugs found when running `io_tests` with more than two locales in `ARKOUDA_RUNNING_MODE=client`

fixes #3783, fixes #3784, fixes #3788

Co-authored-by: Tess Hayes <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stress-tess stress-tess

Labels
bug Something isn't working File IO Arkouda file IO capabilities
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes #3783, #3784, #3788: multilocale io test failures stress-tess/arkouda
1 participant
@stress-tess