tustvold opened a new pull request, #1980:
URL: https://github.com/apache/arrow-rs/pull/1980

   # Which issue does this PR close?
   
   Closes #1978 
   Part of #1979 
   
   # Rationale for this change
    
   The finish_dict method has always struck me as a little bit strange, and it 
is definitely not safe. This PR just removes it
   
   # What changes are included in this PR?
   
   Starts the process of #1979 and uses this to remove the need for 
`PrimitiveBuilder::finish_dict`
   
   # Are there any user-facing changes?
   
   Yes, this removes an API
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to