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

✨ Introduce log environment #679

Merged
merged 2 commits into from
Jan 13, 2025
Merged

✨ Introduce log environment #679

merged 2 commits into from
Jan 13, 2025

Conversation

elbeno
Copy link
Contributor

@elbeno elbeno commented Jan 12, 2025

Problem:

  • Logging is parameterized on many axes. Most logging systems include the level.
    Some also include a module or source. Some include other parameters.
  • Logging machinery should be agnostic to the parameterizations of each logging
    backend. Adding extra parameters to logging macros is not scalable.

Solution:

  • Expand the current logging module to the idea of a logging environment. An
    environment allows compile-time scope-based arguments to be passed through the
    logging machinery; logging backends can extract whichever values they can use
    from the current environment.

Problem:
- `stdx` added a breaking change, viz. the ct_string format result always being
  returned from `ct_format`.

Solution:
- Update code to deal with that.
@elbeno elbeno force-pushed the log-env branch 2 times, most recently from 0d5762e to c887b00 Compare January 12, 2025 05:31
Problem:
- Logging is parameterized on many axes. Most logging systems include the level.
  Some also include a module or source. Some include other parameters.
- Logging machinery should be agnostic to the parameterizations of each logging
  backend. Adding extra parameters to logging macros is not scalable.

Solution:
- Expand the current logging module to the idea of a logging environment. An
  environment allows compile-time scope-based arguments to be passed through the
  logging machinery; logging backends can extract whichever values they can use
  from the current environment.
Copy link
Contributor

@lukevalenty lukevalenty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥

@lukevalenty lukevalenty merged commit 5726ca9 into intel:main Jan 13, 2025
22 checks passed
@elbeno elbeno deleted the log-env branch January 17, 2025 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants