Commit cc72590
[ZEPPELIN-6240] Use generics in CommandArgs constructor to improve type safety
### What is this PR for?
As <at>ParkGyeongTae mentioned,
in the file zeppelin/file/src/main/java/org/apache/zeppelin/file/FileInterpreter.java,
the variables args and flags were originally created using raw types:
```
args = new ArrayList();
flags = new HashSet();
```
To resolve compiler warnings and ensure proper type checking,
I updated the code to use generic types with diamond operators (<>) like this:
```
args = new ArrayList<String>();
flags = new HashSet<String>();
```
### What type of PR is it?
Refactoring
### Todos
* [x] - Use generics in CommandArgs constructor
### What is the Jira issue?
* Open an issue on [Jira](https://issues.apache.org/jira/browse/ZEPPELIN-6240)
### How should this be tested?
* Strongly recommended: add automated unit tests for any new or changed behavior
* Outline any manual steps to test the PR here.
### Screenshots (if appropriate)
### Questions:
* Does the license files need to update? - No
* Is there breaking changes for older versions? - No
* Does this needs documentation? - No
Closes #4971 from hyunw9/feat/Use-generics-in-CommandArgs-constructor.
Signed-off-by: Philipp Dallig <philipp.dallig@gmail.com>
(cherry picked from commit 91f091e)
Signed-off-by: Philipp Dallig <philipp.dallig@gmail.com>1 parent 72525d2 commit cc72590
1 file changed
Lines changed: 2 additions & 2 deletions
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
64 | 64 | | |
65 | 65 | | |
66 | 66 | | |
67 | | - | |
68 | | - | |
| 67 | + | |
| 68 | + | |
69 | 69 | | |
70 | 70 | | |
71 | 71 | | |
| |||
0 commit comments