gianm commented on issue #8814: SQL: Add RAND() function. URL: https://github.com/apache/incubator-druid/pull/8814#issuecomment-550681737 Actually, I just realized this is a bad idea, for two reasons. 1. The result of the random expression will be potentially cached at query time. Fixing this means adding an `isCacheable` method to all expressions. 2. At ingest time, if you use this in a `transformSpec`, it makes the results of ingestion nondeterministic. This will cause haywire with Kafka index task replicas. Fixing it probably requires creating a per-partition seed ahead of time on the supervisor and sharing it across tasks. These issues can be overcome, but, it doesn't seem worth it at this time. So I will close this PR. Thanks @suneet-amp for taking a look anyway!
---------------------------------------------------------------- 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 --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
