tkaymak commented on PR #38255: URL: https://github.com/apache/beam/pull/38255#issuecomment-4320295493
**Coverage follow-up pushed (`2c05a45`):** `codecov/patch` on `292ebe05` flagged 10 lines across 3 shared-base files. The check is informational (no `coverage.status.patch` rule in `.github/codecov.yml`), but tightened the new branches anyway: 1. **`SparkRunnerKryoRegistrator`** — extracted the Scala 2.12/2.13 `Class.forName` fallback into a `@VisibleForTesting findFirstAvailableClass(String...)` helper; new tests cover first-hit / fallback / no-match / empty-input. 2. **`EvaluationContext`** — new `EvaluationContextTest` with Mockito tests for the `catch` blocks in `evaluate()` and `collect()`, including the null-message path that motivated the `String.valueOf(...)` wrap. The remaining 2 lines are the streaming-portable `.toList()` Scala 2.13 fix, exercised by the Spark Streaming ValidatesRunner PostCommit (not measured by codecov). -- 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]
