basantjamwal opened a new pull request, #51371:
URL: https://github.com/apache/arrow/pull/51371
Closes #51367
### Rationale for this change
No code within Arrow uses `arrow::internal::LruCache`. Per discussion in
#51328 (review comment r4027861697), removing this dead code along with
its tests and benchmarks.
### What changes are included in this PR?
- Removed cpp/src/arrow/util/cache_internal.h (LruCache, MemoizeLru,
MemoizeLruThreadUnsafe — all unused outside this file and its tests)
- Removed cpp/src/arrow/util/cache_test.cc
- Removed cpp/src/arrow/util/cache_benchmark.cc
- Removed corresponding entries from cpp/src/arrow/util/CMakeLists.txt
Verified via `git grep` across cpp/ that nothing else references this
code (gandiva's separate LRU cache in cpp/src/gandiva/lru_cache.h is
unrelated and untouched).
### Are these changes tested?
Pure removal of unused code; no new tests needed. Relying on CI to
validate the build/test suite still passes.
### Are there any user-facing changes?
No.
### Was AI used for this PR?
Yes — I used Claude (Anthropic) to help identify the correct files to
remove, verify no remaining references via grep, and draft this PR
description. I reviewed and verified all changes myself before committing.
**PR code and description written by:**
- [x] Human
- [x] AI
**Reviewed before submission by:**
- [x] Human
- [ ] AI
- [ ] Not reviewed
--
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]
