eunhwa99 opened a new pull request, #4970: URL: https://github.com/apache/zeppelin/pull/4970
### What is this PR for? This PR removes unused overloaded constructors in the DownloadRequest class located at: zeppelin/zeppelin-test/src/main/java/org/apache/zeppelin/test/DownloadRequest.java. Specifically, the constructor: ``` public DownloadRequest(URL url, int retries) ``` is not used anywhere within the Zeppelin codebase or its test suite. Removing these unused constructors simplifies the class and reduces maintenance complexity, without affecting any functional behavior. As this constructor was removed, other constructors that were only used by it have also been removed accordingly. ### What type of PR is it? Refactoring ### Todos * [ ] - Verify internal and external usage * [ ] - Remove unused constructors ### What is the Jira issue? * Open an issue on Jira. [ZEPPELIN-6220](https://issues.apache.org/jira/browse/ZEPPELIN/) ### 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? * Is there breaking changes for older versions? * Does this needs documentation? -- 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]
