Page MenuHomePhorge
Differential D4576

Better approach to log SQL queries
ClosedPublic
Actions

Authored by machniak on Nov 23 2023, 2:48 PM.
Tags
None
Referenced Files
F12990825: D4576.diff
Wed, Jun 26, 2:46 AM
Unknown Object (File)
Sun, Jun 16, 8:45 PM
Unknown Object (File)
Sun, Jun 16, 5:04 PM
Unknown Object (File)
Sun, Jun 16, 10:37 AM
Unknown Object (File)
Sun, Jun 16, 6:34 AM
Unknown Object (File)
Sat, Jun 15, 10:31 PM
Unknown Object (File)
Sat, Jun 8, 8:41 AM
Unknown Object (File)
Sat, Jun 8, 8:09 AM
View All 82 Files
Subscribers
mollekopf
Restricted Project

Details

Reviewers
mollekopf
Group Reviewers
Restricted Project
Commits
rK3bc86f99af50: Better approach to log SQL queries
Summary

It's a more clean solution, and allows to log transaction queries

Test Plan

./phpunit

Diff Detail

Repository
rK kolab
Branch
dev/sql-debug-listener
Lint
Lint Skipped
Unit
No Test Coverage
Build Status
Buildable 45817
Build 17716: arc lint + arc unit

Event Timeline

machniak requested review of this revision.Nov 23 2023, 2:48 PM
machniak created this revision.
Harbormaster completed remote builds in B45817: Diff 13141.Nov 23 2023, 2:48 PM
machniak updated this revision to Diff 13144.Nov 23 2023, 3:01 PM
  • Use event subscriber feature, cleanup
Harbormaster completed remote builds in B45820: Diff 13144.Nov 23 2023, 3:01 PM
machniak updated this revision to Diff 13147.Nov 23 2023, 3:04 PM
  • Add missing app.debug check
Harbormaster completed remote builds in B45823: Diff 13147.Nov 23 2023, 3:04 PM
mollekopf subscribed.Nov 24 2023, 1:19 PM
mollekopf added inline comments.
src/app/Providers/EventServiceProvider.php
19

This seems unrelated?

mollekopf accepted this revision.Nov 24 2023, 1:24 PM
This revision is now accepted and ready to land.Nov 24 2023, 1:24 PM
machniak added inline comments.Nov 25 2023, 4:37 PM
src/app/Providers/EventServiceProvider.php
19

This seems unrelated?

Right, This is unused code I cleaned up.

mollekopf added inline comments.Nov 28 2023, 11:57 AM
src/app/Providers/EventServiceProvider.php
19

Ok, I'd prefer unrelated things in a separate commit (doesn't require a separate review, or any review for cases like this). Doesn't really matter for this case.

Closed by commit rK3bc86f99af50: Better approach to log SQL queries (authored by machniak). · Explain WhyNov 29 2023, 11:35 AM
This revision was automatically updated to reflect the committed changes.
machniak added a commit: rK3bc86f99af50: Better approach to log SQL queries.

Revision Contents
Changeset List

PathSize
src/
app/
Listeners/
85 lines
Providers/
36 lines
13 lines

Diff 13141

View Options

src/app/Listeners/SqlDebug.php

Loading...
View Options

src/app/Providers/AppServiceProvider.php

Loading...
View Options

src/app/Providers/EventServiceProvider.php

Loading...