deniskuzZ commented on PR #8202: URL: https://github.com/apache/iceberg/pull/8202#issuecomment-4661069208
> There are alternative histogram sketches to KLL, with various pros and cons. Would it make sense to make this PR a bit more general, so that the type of the sketch is not hardcoded? It would be nice if the engine could decide which histogram algorithm it wants to use. @thomasrebele, apache-datasketches-kll-v1 is itself a sketch type. If the sketch type isn't encoded, how would we determine what algorithm the sketch represents? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
