Skip to content

Commit 9e0f959

Browse files
kiszksrowen
authored andcommitted
[SPARK-23997][SQL][FOLLOWUP] Update exception message
## What changes were proposed in this pull request? This PR is an follow-up PR of apache#21087 based on [a discussion thread](apache#21087 (comment)]. Since apache#21087 changed a condition of `if` statement, the message in an exception is not consistent of the current behavior. This PR updates the exception message. ## How was this patch tested? Existing UTs Closes apache#22269 from kiszk/SPARK-23997-followup. Authored-by: Kazuaki Ishizaki <[email protected]> Signed-off-by: Sean Owen <[email protected]>
1 parent 3c67cb0 commit 9e0f959

File tree

1 file changed

+2
-2
lines changed
  • sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog

1 file changed

+2
-2
lines changed

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -178,8 +178,8 @@ case class BucketSpec(
178178

179179
if (numBuckets <= 0 || numBuckets > conf.bucketingMaxBuckets) {
180180
throw new AnalysisException(
181-
s"Number of buckets should be greater than 0 but less than bucketing.maxBuckets " +
182-
s"(`${conf.bucketingMaxBuckets}`). Got `$numBuckets`")
181+
s"Number of buckets should be greater than 0 but less than or equal to " +
182+
s"bucketing.maxBuckets (`${conf.bucketingMaxBuckets}`). Got `$numBuckets`")
183183
}
184184

185185
override def toString: String = {

0 commit comments

Comments
 (0)