ruanwenjun commented on PR #14661: URL: https://github.com/apache/dolphinscheduler/pull/14661#issuecomment-1657043350
> Hi @ruanwenjun , I thought again, and suggest keep current fix in this PR and do not remove NONE type. We have the check in `checkResourceUploadStartupState()` for resources CURD and can return message `STORAGE_NOT_STARTUP` when setting NONE type. It's more user-friendly, in addition, use LOCAL type is kind of confused on my side. One solution is we can throw `STORAGE_NOT_STARTUP` exception in NoneStorage. Another solution is we use DEFAULT as default, and open storage is must, we don't support use `NONE`. -- 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]
