srowen commented on a change in pull request #32498:
URL: https://github.com/apache/spark/pull/32498#discussion_r634400887
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
##########
@@ -789,6 +797,39 @@ case class Range(
}
}
+ private def computeHistogramStatistics() = {
Review comment:
Trivial: you could put a return type here.
Is there any concern with the overhead of computing the histogram? It only
happens when enabled of course. I just wonder if we need to micro-optimize the
loop below with something more performant if this is critical, but maybe it
isn't.
--
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]
