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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
