hyunw9 opened a new pull request, #4971: URL: https://github.com/apache/zeppelin/pull/4971
### What is this PR for? As @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 * [ ] - Use generics in CommandArgs constructor ### What is the Jira issue? * Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/ * Put link [here](https://issues.apache.org/jira/secure/RapidBoard.jspa?rapidView=632&projectKey=ZEPPELIN&view=detail&selectedIssue=ZEPPELIN-6240), and add [ZEPPELIN-*Jira number*] in PR title, eg. [ZEPPELIN-533] ### 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 -- 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: reviews-unsubscr...@zeppelin.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
