[GitHub] [spark] MaxGekk commented on pull request #40195: [SPARK-42553][SQL] Ensure at least one time unit after "interval"

2023-03-02 Thread via GitHub
MaxGekk commented on PR #40195: URL: https://github.com/apache/spark/pull/40195#issuecomment-1451827970 > Do you mean launch a new PR to branch-3.3? Here it is: https://github.com/apache/spark/pull/40253 Yep. Thank you. -- This is an automated message from the Apache Git Service. T

[GitHub] [spark] MaxGekk commented on pull request #40195: [SPARK-42553][SQL] Ensure at least one time unit after "interval"

2023-03-01 Thread via GitHub
MaxGekk commented on PR #40195: URL: https://github.com/apache/spark/pull/40195#issuecomment-1451372811 @jiang13021 The changes causes some conflicts in branch-3.3. Could you open a separate PR with a backport to Spark 3.3. -- This is an automated message from the Apache Git Service. To r

[GitHub] [spark] MaxGekk commented on pull request #40195: [SPARK-42553][SQL] Ensure at least one time unit after "interval"

2023-03-01 Thread via GitHub
MaxGekk commented on PR #40195: URL: https://github.com/apache/spark/pull/40195#issuecomment-1451372140 @jiang13021 Congratulations with the first contribution to Apache Spark! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [spark] MaxGekk commented on pull request #40195: [SPARK-42553][SQL] Ensure at least one time unit after "interval"

2023-03-01 Thread via GitHub
MaxGekk commented on PR #40195: URL: https://github.com/apache/spark/pull/40195#issuecomment-1451370644 +1, LGTM. Merging to master/3.4. Thank you, @jiang13021. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR