zeroshade opened a new pull request, #362:
URL: https://github.com/apache/arrow-go/pull/362
### Rationale for this change
#326 moved to using `atomic.Int64{]` for the refcount, as a result we found
many places that we were copying the value. For linting reasons we should
remove those.
### What changes are included in this PR?
Utilizing generics to clean up the dictionary array code significantly,
along with removing most of the cases where we were copying the refcount which
contains a lock.
### Are these changes tested?
Yes, the existing unit tests cover it all.
### Are there any user-facing changes?
Nope.
--
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-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
