reuvenlax commented on issue #11034: [BEAM-9424] Allow grouping by LogicalType URL: https://github.com/apache/beam/pull/11034#issuecomment-595347570 IT's not just performance critical code. We wanted these transforms to be usable by code that doesn't understand the logical type. For example: you should be able to write a SQL expression over a type that contains a OneOf. SQL doesn't understand the OneOf logical type of course, however the base type is a Row with nullable fields, Since today getValue returns the base type, you should be able to query it. Similar story with Enumerations - you should be able to treat them as integers in a SQL statement or in one of the schema transforms. (note: I"m not sure if this has been tested with SQL, so it may not work today). We need to figure out a good API that allows these transformation to use the base type by default but _also_ allows you to write a combiner that accepts the appropriate value type. Reuven On Thu, Mar 5, 2020 at 9:12 AM Gleb Kanterov <[email protected]> wrote: > @reuvenlax <https://github.com/reuvenlax> is there an option to have > alternative to attachValues, but for getting values to be used in > performance-critical code? > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/apache/beam/pull/11034?email_source=notifications&email_token=AFAYJVJTSD3CR6X326ZZTCLRF7MQPA5CNFSM4LAVOCP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEN6DQJI#issuecomment-595343397>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/AFAYJVMJPB2WUZ2G6MLDOTLRF7MQPANCNFSM4LAVOCPQ> > . >
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
