pan3793 commented on a change in pull request #2214:
URL: https://github.com/apache/incubator-kyuubi/pull/2214#discussion_r834973396
##########
File path:
integration-tests/kyuubi-kubernetes-it/src/test/scala/org/apache/kyuubi/kubernetes/test/deployment/KyuubiOnKubernetesTestsSuite.scala
##########
@@ -15,13 +15,23 @@
* limitations under the License.
*/
-package org.apache.kyuubi.kubernetes.test
+package org.apache.kyuubi.kubernetes.test.deployment
import org.apache.kyuubi.Logging
+import org.apache.kyuubi.kubernetes.test.MiniKube
import org.apache.kyuubi.operation.SparkQueryTests
-// TODO: [KYUUBI-863] Support test Spark engine using k8s master with minikube
-class KubernetesJDBCTestsSuite extends SparkQueryTests with Logging {
+/**
+ * This test is for Kyuubi Server on Kubernetes with Spark engine:
+ *
+ * Real World Kubernetes Pod (docker container)
Review comment:
I suggest to remove `(docker container)` in description, because a Pod
is compose of one or more containers, and K8s has dropped the support of
DockerShim
--
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]
