Add CREATE DATABASE DDL with Deployer mechanism for K8s Database objects

[ryannedolan]

Summary

• Added create database ddl

Details

Introduces CREATE [OR REPLACE] DATABASE <name> [WITH (...)] syntax that deploys a Database CRD to Kubernetes via the Deployer pattern. Includes parser, executor, shared processCreateDatabase util, K8sDatabaseDeployer, and quidem test with !specify support.

Testing Done

> create database "mydb" WITH (url 'jdbc:mysql://localhost:3306/mydb', driver 'com.mysql.cj.jdbc.Driver', schema 'myschema', dialect 'MySQL');

produces:

apiVersion: hoptimator.linkedin.com/v1alpha1
kind: Database
metadata:
  name: mydb
spec:
  dialect: MySQL
  driver: com.mysql.cj.jdbc.Driver
  schema: myschema
  url: jdbc:mysql://localhost:3306/mydb

[github-actions]

Code Coverage

Overall Project	84.33% -0.23%	🟢
Files changed	72.4%	🟢

File	Coverage
K8sDatabaseDeployer.java	100%	🟢
K8sDeployerProvider.java	100%	🟢
HoptimatorDdlExecutor.java	92.71% -3.26%	❌
HoptimatorDdlUtils.java	85.34% -1.73%	🟢
DatabaseDeployable.java	78.95% -21.05%	🟢

[jogrogan]

lgtm but please add unit tests for any non-generated parser code. There should be existing example for unit testing deployers and HoptimatorDdlUtils and whatnot

[ryannedolan]

@copilot can you add the unit tests referenced above

[Copilot]

lgtm but please add unit tests for any non-generated parser code. There should be existing example for unit testing deployers and HoptimatorDdlUtils and whatnot

Added unit tests in commit be4a9ba:

• K8sDatabaseDeployerTest (11 tests) — covers toK8sObject() behaviors: URL required, case-insensitive option lookup (the d8e73a1 fix), all options populated (driver/schema/catalog/dialect), dialect enum validation, API interactions, YAML output format, and name canonicalization.
• processCreateDatabase tests in HoptimatorDdlUtilsTest (5 tests) — covers compound name rejection, SPECIFY mode empty specs, viewPath content, no-options handling, and specifyFromSql routing.
• K8sDeployerProviderTest (1 test) — verifies DatabaseDeployable routes to K8sDatabaseDeployer.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• checkstyle.org
  • Triggering command: /opt/hostedtoolcache/CodeQL/2.25.1/x64/codeql/tools/linux64/java/bin/java /opt/hostedtoolcache/CodeQL/2.25.1/x64/codeql/tools/linux64/java/bin/java -jar /opt/hostedtoolcache/CodeQL/2.25.1/x64/codeql/xml/tools/xml-extractor.jar --fileList=/tmp/codeql-scratch-c2bb8f7fb6eb8b71/dbs/java/working/files-to-index10062266869957039771.list --sourceArchiveDir=/tmp/codeql-scratch-c2bb8f7fb6eb8b71/dbs/java/src --outputDir=/tmp/codeql-scratch-c2bb8f7fb6eb8b71/dbs/java/trap/java (dns block)
• linkedin.jfrog.io
  • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED -XX:MaxMetaspaceSize=384m -XX:+HeapDumpOnOutOfMemoryError -Xms256m -Xmx512m -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[ryannedolan]

@copilot fix checkstyle errors:

Error: eckstyle] [ERROR] /home/runner/work/Hoptimator/Hoptimator/hoptimator-k8s/src/test/java/com/linkedin/hoptimator/k8s/K8sDatabaseDeployerTest.java:6:8: Unused import - com.linkedin.hoptimator.k8s.models.V1alpha1DatabaseSpec. [UnusedImports]

[Copilot]

@copilot fix checkstyle errors:

...

Fixed in commit 7c43c74 — removed the unused V1alpha1DatabaseSpec import.

[jogrogan]

LGTM