You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "fallintoplace (via GitHub)" <gi...@apache.org> on 2026/06/30 22:35:27 UTC

[PR] Clear dictionary data on reset [arrow-go]

fallintoplace opened a new pull request, #876:
URL: https://github.com/apache/arrow-go/pull/876

   ### What changed
   
   Data.Reset now releases and clears any existing dictionary data when reusing an ArrayData value.
   
   ### Why
   
   Reset replaces the buffers, children, type, length, null count, and offset, but previously left the old dictionary attached. Reusing dictionary-backed data as non-dictionary data could keep returning stale dictionary data and retain the old dictionary longer than expected.
   
   ### Validation
   
   - go test ./arrow/array -run 'TestDataReset' -count=1
   - go test ./arrow/array -count=1
   - PARQUET_TEST_DATA=/Users/hoangvu/Code/OSS/arrow-go/parquet-testing/data ARROW_TEST_DATA=/Users/hoangvu/Code/OSS/arrow-go/arrow-testing/data go test ./... -count=1


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] fix(array): clear dictionary data on reset [arrow-go]

Posted by "zeroshade (via GitHub)" <gi...@apache.org>.
zeroshade commented on PR #876:
URL: https://github.com/apache/arrow-go/pull/876#issuecomment-4857941756

   just needs a rebase


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] fix(array): clear dictionary data on reset [arrow-go]

Posted by "zeroshade (via GitHub)" <gi...@apache.org>.
zeroshade merged PR #876:
URL: https://github.com/apache/arrow-go/pull/876


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org