Skip to content

feat: logger exporter ConsoleLogRecordExporter and InMemoryLogRecordExporter #211

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
Open
wants to merge 57 commits into
base: master
Choose a base branch
from
Open

feat: logger exporter ConsoleLogRecordExporter and InMemoryLogRecordExporter #211

wants to merge 57 commits into from

Conversation

yuzurihaaa
Copy link
Contributor

Which problem is this PR solving?

Fixes # (issue)

Short description of the change

  1. This is continuation from feat: logger processor #209, please review the mentioned PR before this one as the intention is to make the PR as small as possible
  2. This PR mainly address exporter part of Opentelemetry Logs, on ConsoleLogRecordExporter and InMemoryLogRecordExporter
  3. I purposely create exclude OTLP from this PR so that the next PR to do OTLP will only focus on that.

How Has This Been Tested?

  1. Unit test

Checklist:

  • Unit tests have been added
  • Documentation has been updated

@yuzurihaaa yuzurihaaa mentioned this pull request Jan 20, 2025
Open
2 tasks
@yuzurihaaa yuzurihaaa force-pushed the feat/logger-exporter branch from 343bd07 to e1bf474 Compare January 20, 2025 16:55
@yuzurihaaa yuzurihaaa force-pushed the feat/logger-exporter branch from e1bf474 to 78e3924 Compare March 3, 2025 02:27
@yuzurihaaa yuzurihaaa force-pushed the feat/logger-exporter branch 2 times, most recently from 760ff05 to 2bb6ac1 Compare March 18, 2025 05:07
yuzurihaaa added 22 commits March 18, 2025 13:12
@yuzurihaaa
feat: LogRecord limits
dbda356
@yuzurihaaa
feat: LogRecord
630f2cf
@yuzurihaaa
chore: fix comment
7119f8e
@yuzurihaaa
chore: log record limit test
6fb8422
@yuzurihaaa
feat: LogRecordProcessor
69e4e18
@yuzurihaaa
feat: NoopLogRecordProcessor
79640fb
@yuzurihaaa
feat: LoggerConfig
8ff6c71
@yuzurihaaa
feat: Logger
d3d98e6
@yuzurihaaa
feat: LoggerProvider
f4b05aa
@yuzurihaaa
feat: exports
ff5d1e3
@yuzurihaaa
wip: change processors initialization from const
1e050b4
@yuzurihaaa
wip: fix test
5eed62e
@yuzurihaaa
wip: fix import arrange
99e374f
@yuzurihaaa
wip: attach copyright
38b2137
@yuzurihaaa
wip: change from api.SpanContext? and api.Context? to non-null pr…
88eb916 
…ivate `api.Context` with `api.Context?` as constructor argument. If no `api.Context` is passed into constructor, `api.Context.current` is used instead.
@yuzurihaaa
wip: remove unused includeTraceContext
014f087
@yuzurihaaa
wip: opentelemetry as defaultLoggerName
96938b8
@yuzurihaaa
wip: make log processors immutable
2a9a80a
@yuzurihaaa
wip: make all variables private
f0206c6
@yuzurihaaa
wip: update limit to only create if value changes.
01636f9
@yuzurihaaa
wip: change from sdk.Attributes to List<api.Attribute>
aae1c3e
@yuzurihaaa
wip: change to DateTime
464a636
@yuzurihaaa yuzurihaaa force-pushed the feat/logger-exporter branch from 2bb6ac1 to efb46dc Compare March 18, 2025 05:13
@evanweible-wf evanweible-wf removed their request for review April 25, 2025 16:18
@yuzurihaaa yuzurihaaa force-pushed the feat/logger-exporter branch from efb46dc to f81e28e Compare May 9, 2025 03:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@blakeroberts-wk blakeroberts-wk Awaiting requested review from blakeroberts-wk blakeroberts-wk is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
Merge Requirements Unmet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yuzurihaaa @rmconsole-wf