This is an automated email from the ASF dual-hosted git repository.

yangjie01 pushed a commit to branch branch-3.5
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.5 by this push:
     new 81a939d375b [SPARK-44552][SQL] Remove `private object ParseState` 
definition from `IntervalUtils`
81a939d375b is described below

commit 81a939d375bad482333a8e87f00d21dd64048b0e
Author: yangjie01 <yangji...@baidu.com>
AuthorDate: Fri Jul 28 18:09:48 2023 +0800

    [SPARK-44552][SQL] Remove `private object ParseState` definition from 
`IntervalUtils`
    
    ### What changes were proposed in this pull request?
    SPARK-44326(https://github.com/apache/spark/pull/41885) moved the relevant 
code from `IntervalUtils` to `SparkIntervalUtils` (including `private object 
ParseState` definition),
    
    
https://github.com/apache/spark/blob/6ca45c52b7416e7b3520dc902cb24f060c7c72dd/sql/api/src/main/scala/org/apache/spark/sql/catalyst/util/SparkIntervalUtils.scala#L247-L260
    
    but did not delete the definition of `private object ParseState` in 
`IntervalUtils`. So this pr cleans up it.
    
    ### Why are the changes needed?
    Code cleanup.
    
    ### Does this PR introduce _any_ user-facing change?
    No
    
    ### How was this patch tested?
    Pass Github Actions
    
    Closes #42165 from LuciferYang/SPARK-44552.
    
    Lead-authored-by: yangjie01 <yangji...@baidu.com>
    Co-authored-by: YangJie <yangji...@baidu.com>
    Signed-off-by: yangjie01 <yangji...@baidu.com>
    (cherry picked from commit 7ee9e43ec6891f26b36a090a83536d3cae682861)
    Signed-off-by: yangjie01 <yangji...@baidu.com>
---
 .../apache/spark/sql/catalyst/util/IntervalUtils.scala    | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git 
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/IntervalUtils.scala
 
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/IntervalUtils.scala
index 6ba59b4e730..24620e692a0 100644
--- 
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/IntervalUtils.scala
+++ 
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/IntervalUtils.scala
@@ -739,21 +739,6 @@ object IntervalUtils extends SparkIntervalUtils {
     fromDoubles(interval.months / num, interval.days / num, 
interval.microseconds / num)
   }
 
-  private object ParseState extends Enumeration {
-    type ParseState = Value
-
-    val PREFIX,
-        TRIM_BEFORE_SIGN,
-        SIGN,
-        TRIM_BEFORE_VALUE,
-        VALUE,
-        VALUE_FRACTIONAL_PART,
-        TRIM_BEFORE_UNIT,
-        UNIT_BEGIN,
-        UNIT_SUFFIX,
-        UNIT_END = Value
-  }
-
   /**
    * A safe version of `stringToInterval`. It returns null for invalid input 
string.
    */


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to