benibus opened a new issue, #35437: URL: https://github.com/apache/arrow/issues/35437
### Describe the enhancement requested Several of the `DictionaryArray` accessor methods (`dictionary()` and `indices()`) currently return a `std::shared_ptr<Arrray` by value. From looking at the implementation, these underlying arrays are owned by the class so it seems possible to return them as a `const std::shared_ptr<Array>&` instead - keeping with general conventions and reducing potential gotchas. ### Component(s) C++ -- 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]
