apache / apache/texera

Remove the unused TupleUtils.json2tuple

Open
#8,392 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

`TupleUtils.json2tuple` has no production caller. Its sibling `tuple2json` is live (`ExecutionResultService` uses it) and stays.

**Introduced by** #1328 (2021-09-08, "[Operator Caching Step 3] add operator cache backend new files and config changes"), and genuinely used — the cache read path called `val newTuple = json2tuple(line)` when rehydrating cached tuples.

**Usage removed by** #3111 (2024-11-26, "Update amber to depend on sub projects"), which deleted that call site while splitting amber into sub-projects. From that commit on, the only references are the definition and its tests.

Dead for about two years. Removing it also frees three imports that only it used: `AttributeTypeUtils.{inferSchemaFromRows, parseField}`, `JSONUtils.{JSONToMap, objectMapper}`, and `ArrayBuffer`.

Pure deletion, no behaviour change: −114 lines.

### Task Type

- [x] Refactor / Cleanup

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.