Fix typo: AGGREGRATE_TIME_LIST → AGGREGATE_TIME_LIST#17171
Open
PDGGK wants to merge 1 commit intoapache:masterfrom
Open
Fix typo: AGGREGRATE_TIME_LIST → AGGREGATE_TIME_LIST#17171PDGGK wants to merge 1 commit intoapache:masterfrom
PDGGK wants to merge 1 commit intoapache:masterfrom
Conversation
Fixes apache#17155 - AbstractCli.java: Rename constant from AGGREGRATE_TIME_LIST to AGGREGATE_TIME_LIST - Cli.java: Update all references to use correct spelling Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Pull request overview
This PR fixes a spelling typo in a constant name, changing AGGREGRATE_TIME_LIST to AGGREGATE_TIME_LIST throughout the CLI codebase. The typo was correctly identified in issue #17155, and this change ensures the constant name properly reflects its purpose of storing time aggregation operations.
Changes:
- Renamed the constant from
AGGREGRATE_TIME_LISTtoAGGREGATE_TIME_LIST(fixing the misspelling of "aggregate") - Updated all usages of the constant in the codebase
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| AbstractCli.java | Renamed the constant definition from AGGREGRATE_TIME_LIST to AGGREGATE_TIME_LIST |
| Cli.java | Updated both references (lines 188, 203) to use the corrected constant name |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
2 tasks
HTHou
reviewed
Feb 5, 2026
| static final int CODE_ERROR = 1; | ||
|
|
||
| static final String ISO8601_ARGS = "disableISO8601"; | ||
| static final List<String> AGGREGRATE_TIME_LIST = new ArrayList<>(); |
Contributor
There was a problem hiding this comment.
Looks like this list is useless. Better to remove it?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fix typo in constant name:
AGGREGRATE_TIME_LIST→AGGREGATE_TIME_LISTFixes #17155
Changes
AbstractCli.java: Rename constant fromAGGREGRATE_TIME_LISTtoAGGREGATE_TIME_LISTCli.java: Update all references (lines 188, 203) to use correct spellingNotes
🤖 Generated with Claude Code