aokolnychyi commented on a change in pull request #30577:
URL: https://github.com/apache/spark/pull/30577#discussion_r535014598
##########
File path:
sql/core/src/test/scala/org/apache/spark/sql/SparkSessionExtensionSuite.scala
##########
@@ -88,6 +88,12 @@ class SparkSessionExtensionSuite extends SparkFunSuite {
}
}
+ test("inject data source rewrite rule") {
Review comment:
Sure, I can do that. If I remember correctly, Spark used to prepend JIRA
prefixes in test names only for bug fixes. I think I was asked to remove such
prefixes one or two times before. I believe there was a recent discussion on
the dev list regarding this point but I don't recall the summary. Could you
fill me in on the current strategy, @HyukjinKwon? The guide says there should
be JIRA prefix every time we add more tests to existing suites. Do I understand
that correctly?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
