davidm-db commented on code in PR #54905:
URL: https://github.com/apache/spark/pull/54905#discussion_r3136864815
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/types/ops/TimeTypeOps.scala:
##########
@@ -48,7 +54,8 @@ import org.apache.spark.sql.types.ops.TimeTypeApiOps
* The TimeType with precision information
* @since 4.2.0
*/
-case class TimeTypeOps(override val t: TimeType) extends TimeTypeApiOps(t)
with TypeOps {
+case class TimeTypeOps(override val t: TimeType)
+ extends TimeTypeApiOps(t) with TypeOps {
Review Comment:
@MaxGekk linter is failing and scalafmt is correcting me to do more spaces...
<img width="1482" height="416" alt="image"
src="https://github.com/user-attachments/assets/1a2c9ec4-41d3-44e0-83d7-9eec7877b05c";
/>
so I guess there really isn't a choice for this PR, but we should either:
- look into fixing scalafmt - this probably has a lot of consequences for
the already existing code...
- or accepting the diff
--
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]
