clintropolis commented on pull request #10086: URL: https://github.com/apache/druid/pull/10086#issuecomment-651532592
>The changes you are suggesting looks good: use double instead of float and fix a typo in a function name but how does that address "fixes missing columns during Transform step"? what missing columns? I think the change is to match the case statement here: https://github.com/apache/druid/blob/02c1beb6ab43896744c02ff38bae124c6551f404/web-console/src/utils/druid-type.ts#L71 which calls the changed method, but 'float' matches nothing so would be lost. Technically there are both float and a double sum aggs, but I'm not sure how you would distinguish between which to use so using 'double' is probably better. ---------------------------------------------------------------- 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]
