apache / apache/texera

Remove the commented-out MockKillWorkerResource

Open
#8,393 0 comments 0 reactions 1 assignee Claimed by @aglinxinyuan View on GitHub
Dominant language
Scala
Stars
314
Forks
187
Avg merge
1d 21h
Merged PRs (30d)
214

Description

### Task Summary

`MockKillWorkerResource.scala` is a file whose every non-blank line after the ASF header is a comment — the commented-out remains of a JAX-RS `/kill/worker` endpoint. `TexeraWebApplication` still carries the matching commented-out registration line, `// environment.jersey().register(classOf[MockKillWorkerResource])`.

**Introduced by** the initial amber import, commit `41a8a92017` (2020-08-20) — predates the PR workflow — which also registered it live with Jersey.

**Disabled by** #1000 (`4a64935c35`, 2021-01-30, "Migrate all control handling logics to AsyncRPC and cleanup Amber"), which commented out the registration line and the whole file body in the same diff. Nothing has compiled from the file since.

Registered live for about five months, 2020-08-20 → 2021-01-30, then commented out for the five and a half years since. The commented registration line goes with the file.

Pure deletion, no behaviour change: −40 lines.

### Task Type

- [x] Refactor / Cleanup
- [ ] DevOps / Deployment / CI
- [ ] Testing / QA
- [ ] Documentation
- [ ] Performance
- [ ] Other

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.