-
Notifications
You must be signed in to change notification settings - Fork 33
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
Replace convention of created_by being None for system process #92
Comments
A scan through RDM-Records, Requests, and Records-Resources (the latter two after the lift-up refactoring) gave me the following results: RDM-Records
None of the spots mentioned above actually use Records-ResourcesThe Resolver Registry encodes the RequestsWhile requests must have some routing information (a sender and a receiver), it is currently entirely possible for requests to not have a topic (i.e. ConclusionI think that it should still be valid to have a value of Examples: Also, we need to introduce a new Entity Proxy, as well as decide on how to |
There was a convention of using None in the created_by field to mark the creator as the system process. This should be changed to just another regular entity representation for this identity e.g.
{"system": -1}
or some such thing. This will greatly reduce the None checking overhead and uniformize the code base.This change may have to actually be made in a module higher up.
The text was updated successfully, but these errors were encountered: