Github user GJL commented on a diff in the pull request:
https://github.com/apache/flink/pull/4896#discussion_r158261437
--- Diff:
flink-libraries/flink-table/src/test/scala/org/apache/flink/table/runtime/stream/TimeAttributesITCase.scala
---
@@ -18,8 +18,8 @@
package org.apache.flink.table.runtime.stream
-import java.math.BigDecimal
import java.lang.{Integer => JInt, Long => JLong}
+import java.math.BigDecimal
--- End diff --
nit: Are we following import orders for Scala as described here for Java:
http://flink.apache.org/contribute-code.html#imports?
---
