askoa commented on PR #3553: URL: https://github.com/apache/arrow-rs/pull/3553#issuecomment-1399567309
> > * I wonder if we should use `RunArray / RunBuilder` instead of either `RunEndEncoded` or `REE` . The latter is not a standard initialism, and the former is a little odd imo, not to mention verbose, given it isn't `DictionaryEncodedArray`. > > I'll change it to `RunEndsArray`. It is more specific as we use Run End Encoding and not Run Length Encoding. Just to give some additional context. The name `RunEndEncoded` is defined as a Type in fbs file. Whereas `Dictionary` is not defined as a Type. So when defining and array for the Type, `RunEndEncodedArray` is consistent with other Types like `LargeBinary` whose array is `LargeBinaryArray`. Using `RunArray` is inconsistent on how we name array for a type. Does the suggestion include changing the name of data type to `Run` instead of `RunEndEncoded`? https://github.com/apache/arrow/blob/cb0f0178dacd922f355255a16c85820fb4e390c8/format/Schema.fbs#L426-L430 -- 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]
