Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/22989#discussion_r232983941
--- Diff: dev/checkstyle-suppressions.xml ---
@@ -46,4 +46,12 @@
files="sql/catalyst/src/main/java/org/apache/spark/sql/streaming/GroupStateTimeout.java"/>
<suppress checks="MethodName"
files="sql/core/src/main/java/org/apache/spark/sql/streaming/Trigger.java"/>
+ <suppress checks="RegexpSingleline"
--- End diff --
I found the problem is `RegexpSingleline` can't put into `TreeWalker`, so
`SuppressionCommentFilter` rule can't effect on it, `RegexpSinglelineJava` is
the right choice. Also spend some time on comment syntax, maybe the usage
description can be improved a little, please have a check whether I understand
this rule correctly. Done in a4f49ce.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
