Skip to content
This repository was archived by the owner on Apr 26, 2025. It is now read-only.

[DNM] [NP-3798] For debugging, store the path of rules and property changes a UCJ went through #4935

Open
KernelDeimos wants to merge 3 commits into
base: master
Choose a base branch
from
Open

[DNM] [NP-3798] For debugging, store the path of rules and property changes a UCJ went through #4935

KernelDeimos wants to merge 3 commits into from

Conversation

@KernelDeimos
Copy link
Contributor

@KernelDeimos KernelDeimos commented Mar 9, 2021 •
edited
Loading

The following improvements are needed before this can be merged

  • Should not log this data in production (very verbose)
  • It might be logging too much data (turns out it wasn't)
  • Should record when the rule engine started so it's easy to see where one chain ends and another begins
  • Any rule doing a "re-put" should probably record that it occurred and why

@google-cla google-cla bot added the cla: yes label Mar 9, 2021
@KernelDeimos KernelDeimos force-pushed the NP-3798/LogUCJUpdates branch from 1689e18 to 30796df Compare March 10, 2021 15:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

cla: yes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@KernelDeimos