[GitHub] [arrow] danepitkin commented on pull request #35210: GH-35209: [Python] Add a type alias for pa.dictionary(pa.int32(), pa.string())

["danepitkin (via GitHub)" <gi...@apache.org>]

danepitkin commented on PR #35210:
URL: https://github.com/apache/arrow/pull/35210#issuecomment-1546114236

   Sorry to go back on this, but after some more thought I'm a bit hesitant to add composite type string aliases. There's hundreds of possibilities (or more) and there can also be several variations of string representations each (e.g. spaces vs no spaces). We also aren't matching the string representation of `pa.dictionary.__str__` here, which looks something like this: `DictionaryType(dictionary<values=string, indices=int32, ordered=0>)`. 
   
   @jorisvandenbossche @AlenkaF any thoughts here?


-- 
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