Skip to content

Expose plan monitor report paths - #1255

Open
wayyoungboy wants to merge 1 commit into
oceanbase:masterfrom
wayyoungboy:fix-issue-1011-plan-monitor-dir
Open

wayyoungboy wants to merge 1 commit into
oceanbase:masterfrom
wayyoungboy:fix-issue-1011-plan-monitor-dir

Conversation

@wayyoungboy

Copy link
Copy Markdown
Member

Summary

  • print the generated SQL plan monitor HTML report path and resources directory
  • return report_file and resources_dir in the gather result data alongside store_dir
  • add unit coverage for the result path payload

Fixes #1011

Verification

  • PYTHONPATH=. python -m pytest test/gather/test_gather_plan_monitor.py test/common/test_scene.py -q
  • black --check -S -l 256 src/handler/gather/gather_plan_monitor.py test/gather/test_gather_plan_monitor.py
  • python workflow_data/check_py_files.py .
  • git diff --check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Enhancement]: gather plan-monitor support locate directory

1 participant