Mrhs121 commented on PR #18098:
URL:
https://github.com/apache/dolphinscheduler/pull/18098#issuecomment-4319962545
Thanks for the review. I updated this PR to align with the feedback:
1. The log API is now narrowed to a unified logType-based access model,
instead of keeping separate APIs for task log and task output.
2. Task log storage is changed to a shared task_logs_root_path instead of
separate file path fields.
3. TaskLogType is kept lightweight, and the path/file resolution logic is
moved into helper classes.
4. For non-command tasks such as HttpTask, SqlTask, and ProcedureTask, I
added a small shared writer so they still follow the same output logging model
without forcing a larger execution-flow refactor.
5. The schema upgrade scripts are moved to the new 3.4.2 upgrade directory.
6. The frontend is simplified by using dropdown actions for Log / Output
instead of adding extra buttons. such as:
<img width="176" height="185" alt="截屏2026-04-25 23 26 07"
src="https://github.com/user-attachments/assets/14cef93a-37c4-4dcb-919c-a56c06b02302";
/> <img width="173" height="220" alt="截屏2026-04-25 23 26 04"
src="https://github.com/user-attachments/assets/d0f7e504-cac2-45b1-884e-06fda3ed0ca2";
/>
--
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]
