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.

Sign up for GitHub

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

Jump to bottom

Group all PID dependant objects under a single object (Refactor) #198

Open
r1viollet opened this issue Oct 20, 2022 · 2 comments
Open

Group all PID dependant objects under a single object (Refactor) #198

r1viollet opened this issue Oct 20, 2022 · 2 comments

Comments

@r1viollet
Copy link
Collaborator

Description

We are calling many different pid clean up functions instead of grouping everything under a single call.
sanchda pushed a commit that referenced this issue Apr 7, 2023
@r1viollet
Async exporter fixups (#198)
ca69aa4 
- Fix flipflop logic
- Avoid fatal error when we receive a warning from exporter
- Avoid pending status
@r1viollet
Copy link
Collaborator Author

The following PR improves the object model
#400

@r1viollet
Copy link
Collaborator Author

The remaining work involves moving

  • DSO objects to the process object
  • Some of the symbol caching objects

The one drawback is having a process object that has many dependencies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@r1viollet