kunwp1 opened a new pull request, #5201: URL: https://github.com/apache/texera/pull/5201
### What changes were proposed in this PR? Fixes the eight `[warn]`/deprecation messages emitted by `sbt clean compile` on JDK 17: | File | Warning | Fix | | --- | --- | --- | | `common/pybuilder/.../BoundaryValidator.scala` | outer reference in this type test (×2) | Move nested `CompileTimeContext` / `RuntimeContext` case classes to the companion object; the path-dependent macro `Position` becomes a `Pos` type parameter. Call sites in `PythonTemplateBuilder.scala` switch from `validator.X(...)` to `BoundaryValidator.X(...)`. | | `file-service/.../DatasetFileNodeSerializer.java` | uses deprecated API | Swap `scala.collection.JavaConverters.seqAsJavaList` for `scala.jdk.javaapi.CollectionConverters.asJava`. | | `common/workflow-operator/.../TestOperators.scala` | `MySQLSourceOpDesc` deprecated (×2) | Delete the unused `inMemoryMySQLSourceOpDesc` helper and its import (no callers; MySQL source dead since 1.1.0-incubating). | | `amber/.../ExecutionResultService.scala` | inferred existential type | Add `import scala.language.existentials`. | | `amber/.../web/auth/JwtAuth.scala` (used in `ComputingUnitMaster.scala` & `TexeraWebApplication.scala`) | `JwtAuth` object deprecated (×2) | Drop the aspirational `@Deprecated` on the wrapper. `common/auth` doesn't depend on Dropwizard, so amber has no replacement to migrate to until that changes. TODO retained on the object with the migration rationale, so the future cleanup is still tracked. | ### Any related issues, documentation, discussions? Closes #5200 ### How was this PR tested? `sbt clean compile` on Eclipse Adoptium JDK 17.0.19: Before: ``` [warn] BoundaryValidator.scala:78:20: The outer reference in this type test cannot be checked at run time. [warn] BoundaryValidator.scala:86:20: The outer reference in this type test cannot be checked at run time. [info] DatasetFileNodeSerializer.java uses or overrides a deprecated API. [warn] TestOperators.scala:150:6: class MySQLSourceOpDesc in package mysql is deprecated [warn] TestOperators.scala:151:41: class MySQLSourceOpDesc in package mysql is deprecated [warn] ExecutionResultService.scala:441:13: inferred existential type ... should be enabled by making the implicit value scala.language.existentials visible. [warn] ComputingUnitMaster.scala:166:5: object JwtAuth in package auth is deprecated [warn] TexeraWebApplication.scala:137:5: object JwtAuth in package auth is deprecated ``` After: zero `[warn]` lines from `sbt clean compile`; build still succeeds (`[success] Total time: 41 s`). No behavior changes — these are warning cleanups only, so existing test suites cover correctness. No new tests added. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Opus 4.7) -- 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]
