Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r221587545
--- Diff:
core/src/test/scala/org/apache/spark/deploy/security/HadoopDelegationTokenManagerSuite.scala
---
@@ -46,6 +47,7 @@ class HadoopDelegationTokenManagerSuite extends
SparkFunSuite with Matchers {
delegationTokenManager.getServiceDelegationTokenProvider("hadoopfs")
should not be (None)
delegationTokenManager.getServiceDelegationTokenProvider("hbase")
should not be (None)
delegationTokenManager.getServiceDelegationTokenProvider("hive")
should not be (None)
+ delegationTokenManager.getServiceDelegationTokenProvider("kafka")
should not be (None)
--- End diff --
There are other test methods where all the others DT providers are tested
similar to this.
Like `test("using deprecated configurations")` you can extend them with
this line as well (to keep the symmetry).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
