jongyoul commented on PR #4308: URL: https://github.com/apache/zeppelin/pull/4308#issuecomment-1100821129
@huage1994 Sorry for the late review but I wondered why you introduced `RemoteStorageOpertaor`. It looks natural to make an interface and its implementation in Java but I feel like it's only for the test object of `MockRemoteStorageOperator` If it does, how about using mockito instead of making MockRemoteOperator? Rest of them, I'm reading and will finish reviewing them by tomorrow. -- 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: dev-unsubscr...@zeppelin.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org