apache / apache/druid

Break up SeekableStreamIndexTaskRunner.runInternal()

Open
#7,360 5 comments 3 reactions 0 assignees View on GitHub
Area - Streaming Ingestion Refactoring
Dominant language
Java
Stars
14.1k
Forks
3.8k
Avg merge
2d 58m
Merged PRs (30d)
233

Description

This method has 550+ lines of code. Its cyclomatic complexity is 74 (the reasonable limit of cyclomatic complexity for methods is considered 14).

@jihoonson @gianm probably you are the only persons who understand this method. It would probably take days for anybody else to understand it to the level to be able to make any change in it with confidence.

Contributor guide

Open the contributing guide

Research direction

Start by locating SeekableStreamIndexTaskRunner.runInternal(), the entry point named in the issue, and read the surrounding class to understand its responsibilities. The issue provides no file path or tests; completion would require breaking up the 550+ line method while preserving its behavior and reducing its cyclomatic complexity.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
backend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.