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

Append CIVICRM_DEBUG_LOG_QUERY to sql log var #309

Merged
merged 1 commit into from
Oct 5, 2020

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Oct 2, 2020

Same as civicrm/civicrm-core#18471 but for the other affected place

@totten

@civibot
Copy link

civibot bot commented Oct 2, 2020

(Standard links)

@totten
Copy link
Member

totten commented Oct 5, 2020

Looks sensible.

Re-tested with similar procedure as before, but with tighter monitoring of the outputted files, e.g.

rm ~/bknix/build/dmaster/web/sites/default/files/civicrm/ConfigAndLog/*log

env CIVICRM_DEBUG_LOG_QUERY=_test_2 cv flush

ls ~/bknix/build/dmaster/web/sites/default/files/civicrm/ConfigAndLog/

Confirmed that (1) the QUERY DONE statements now appear the alternate query log (_test_2) and (2) no other data is written to the default query log.

@totten totten merged commit 994f174 into civicrm:master Oct 5, 2020
@totten totten deleted the log branch October 5, 2020 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants