FrankChen021 commented on pull request #10949: URL: https://github.com/apache/druid/pull/10949#issuecomment-803932960
> > Hmm, so what I have had in mind to solve this is to be able to determine whether a `RowSignature` should be "finalized" or not in terms of the aggregator types. #9638 added some of the pieces needed for this (`getFinalizedType`, etc) and touches on this idea in the PR description, I just haven't yet got back this core refactoring work, or quite had time to fully think through how to determine when we need the 'finalized' signature or not. > > #10277 also added tracking of the "name" of the complex type on `ColumnCapabilities` (which typically is what populates the `RowSignature`) so that is potentially available to give greater detail than `ValueType.COMPLEX`, but I think the finalized type would be the useful information here. > > That said, I haven't had a look at this PR at all yet. I will try to get to it sometime soon, maybe I will have some ideas while looking over the code. The name of complex type has been set in first/last aggregator https://github.com/apache/druid/pull/10949/files#diff-9fedc71bcede0adcbb1deadbef33e9e1de175ee11209eb4d5d676580104f2c03R231 And I checked the code about how `RowSignature` works today, and found that there's no way to get that name from `RowSignature` because when `RowSignature` is instantiated , that name is not passed to `RowSignature` https://github.com/apache/druid/blob/51d2c61f1cd03f2d3fe396c5bb4738293dc92107/sql/src/main/java/org/apache/druid/sql/calcite/schema/DruidSchema.java#L698 So, is it reasonable to make some changes here to pass the type name as well as its value type to `RowSignature` if its value type is COMPLEX ? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
