Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inspector improvement: page load profiler for services and more #5799

Open
ebruchez opened this issue May 3, 2023 · 6 comments
Open

Inspector improvement: page load profiler for services and more #5799

ebruchez opened this issue May 3, 2023 · 6 comments

Comments

@ebruchez
Copy link
Collaborator

ebruchez commented May 3, 2023

See also #2169 and #3534.

A recurring issue is gaining data about a slow-loading page. In a few concrete cases we've had, slow user-defined services are to blame. But it's hard to know about this without inspecting log files.

We should have a way to load a page in a "profiling" mode. This can be done:

  • with the Form Builder Test mode
  • as well as for a deployed form
    • exactly how, and how to access this information, is TBD

Here is the information we'd like to have:

  • total page load time on the server
  • total time spent in user-defined service calls
    • and which services are running
  • total time spent in Form Runner persistence layer
  • total time spent in Form Runner proper
    • document initialization
    • refreshes
    • actions
    • and possibly more

The result could be shown in two ways:

  • stored in memory, for retrieval by an admin view
  • shown directly as a pane on the loaded page (but only if enabled, of course)
@ebruchez
Copy link
Collaborator Author

ebruchez commented May 3, 2023

As a first step, even just gathering the info and logging it in a compact form could help.

@ebruchez
Copy link
Collaborator Author

ebruchez commented May 9, 2023

I made some improvements to the XForms Inspector. I notice that it has a "Console" mode, which allows dispatching fr-messages events to the inspector and have them show there. This doesn't appear to be used anywhere in our code base.

We could use this feature, or an improvement of it, to output this kind of information.

@ebruchez
Copy link
Collaborator Author

ebruchez commented Jun 1, 2023

Moved this comment to separate issue #5846.

1 similar comment
@ebruchez
Copy link
Collaborator Author

ebruchez commented Jun 1, 2023

Moved this comment to separate issue #5846.

@ebruchez
Copy link
Collaborator Author

ebruchez commented Jun 2, 2023

Moved this comment to separate issue #5845.

@ebruchez ebruchez changed the title Page load profiler for services and more Inspector improvement: page load profiler for services and more Jun 2, 2023
@ebruchez
Copy link
Collaborator Author

ebruchez commented Jun 2, 2023

Also for Form Runner actions, including action syntax!

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

No branches or pull requests

1 participant