search

preinheimer / xhprof

XHGUI is a GUI for the XHProf PHP extension, using a database backend, and pretty graphs to make it easy to use and interpret.
Other
833 stars 185 forks source link

WooCommerce require LONGBLOB #110

Closed killua99 closed 1 year ago

killua99 commented 6 years ago

More than an issue is something to update.

MySQL require LONGBLOB to save data for WooCommerce sites (WordPress plugin that transform a simple blog into a webstore, for those who doesn't know).

Also the server_id is server_id char(3) NOT NULL default 't11', when it need to be longer also.

aik099 commented 6 years ago

MySQL require LONGBLOB to save data for WooCommerce sites (WordPress plugin that transform a simple blog into a webstore, for those who doesn't know).

This means, that you have a lots of profiling data I guess.

Also the server_id is server_id char(3) NOT NULL default 't11', when it need to be longer also.

Any example values that goes in there?

You're welcome to send PR with a fixes.

killua99 commented 6 years ago

My server id is, vm.local

Yes, I'm confuse if this case need a PR, what are your thoughts ?

aik099 commented 6 years ago

If you want to change library code, then this is done via PR creation 😉

There are plenty docs online about how to create PR (pull request) if you haven't done this before.