{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":187188248,"defaultBranch":"main","name":"opentelemetry-specification","ownerLogin":"dynatrace-oss-contrib","currentUserCanPush":false,"isFork":true,"isEmpty":false,"createdAt":"2019-05-17T09:31:12.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/49676351?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1715679221.0","currentOid":""},"activityList":{"items":[{"before":"3f13f954ab9d350beb1f898e2caead60ab7f982b","after":"93e5dd5d23d52ac45a3ddd81c4a809ee3e9d27e1","ref":"refs/heads/clarify-attributes-advisory","pushedAt":"2024-05-14T09:40:29.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"pichlermarc","name":"Marc Pichler","path":"/pichlermarc","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/22105064?s=80&v=4"},"commit":{"message":"Clarify Attributes advisory parameter","shortMessageHtmlLink":"Clarify Attributes advisory parameter"}},{"before":"1055359de31304caeddcc05fc2f8df8f6c78185b","after":"3f13f954ab9d350beb1f898e2caead60ab7f982b","ref":"refs/heads/clarify-attributes-advisory","pushedAt":"2024-05-14T09:36:41.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"pichlermarc","name":"Marc Pichler","path":"/pichlermarc","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/22105064?s=80&v=4"},"commit":{"message":"fixup! Clarify Attributes advisory parameter","shortMessageHtmlLink":"fixup! Clarify Attributes advisory parameter"}},{"before":null,"after":"1055359de31304caeddcc05fc2f8df8f6c78185b","ref":"refs/heads/clarify-attributes-advisory","pushedAt":"2024-05-14T09:33:41.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"pichlermarc","name":"Marc Pichler","path":"/pichlermarc","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/22105064?s=80&v=4"},"commit":{"message":"Clarify Attributes advisory parameter","shortMessageHtmlLink":"Clarify Attributes advisory parameter"}},{"before":"a31752fc58dbc335164618659dae4aa6b0b06368","after":null,"ref":"refs/heads/metrics-requirement-level-to-semconv","pushedAt":"2024-04-07T14:51:07.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"joaopgrassi","name":"Joao Grassi","path":"/joaopgrassi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5938087?s=80&v=4"}},{"before":"ea017373fa834e4db709fc28c8f66814bcb8c47a","after":"a31752fc58dbc335164618659dae4aa6b0b06368","ref":"refs/heads/metrics-requirement-level-to-semconv","pushedAt":"2024-04-05T07:22:33.000Z","pushType":"push","commitsCount":3,"pusher":{"login":"joaopgrassi","name":"Joao Grassi","path":"/joaopgrassi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5938087?s=80&v=4"},"commit":{"message":"Merge branch 'main' into metrics-requirement-level-to-semconv","shortMessageHtmlLink":"Merge branch 'main' into metrics-requirement-level-to-semconv"}},{"before":"a4630b0654b24d10f19378d79811c996ed29a26b","after":"ea017373fa834e4db709fc28c8f66814bcb8c47a","ref":"refs/heads/metrics-requirement-level-to-semconv","pushedAt":"2024-03-29T15:55:09.000Z","pushType":"push","commitsCount":4,"pusher":{"login":"arminru","name":"Armin Ruech","path":"/arminru","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/7052238?s=80&v=4"},"commit":{"message":"Merge branch 'main' into metrics-requirement-level-to-semconv","shortMessageHtmlLink":"Merge branch 'main' into metrics-requirement-level-to-semconv"}},{"before":"6e49905d729f2b5f2bec4b05fd6fb9b66472cde6","after":"a4630b0654b24d10f19378d79811c996ed29a26b","ref":"refs/heads/metrics-requirement-level-to-semconv","pushedAt":"2024-03-28T17:32:23.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"joaopgrassi","name":"Joao Grassi","path":"/joaopgrassi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5938087?s=80&v=4"},"commit":{"message":"Merge branch 'main' into metrics-requirement-level-to-semconv","shortMessageHtmlLink":"Merge branch 'main' into metrics-requirement-level-to-semconv"}},{"before":"b03e4ed417884fbeddf4ba59bd05ad81dd80619b","after":"6e49905d729f2b5f2bec4b05fd6fb9b66472cde6","ref":"refs/heads/metrics-requirement-level-to-semconv","pushedAt":"2024-03-26T22:26:44.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"joaopgrassi","name":"Joao Grassi","path":"/joaopgrassi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5938087?s=80&v=4"},"commit":{"message":"Merge branch 'main' into metrics-requirement-level-to-semconv","shortMessageHtmlLink":"Merge branch 'main' into metrics-requirement-level-to-semconv"}},{"before":null,"after":"b03e4ed417884fbeddf4ba59bd05ad81dd80619b","ref":"refs/heads/metrics-requirement-level-to-semconv","pushedAt":"2024-03-22T16:04:31.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"joaopgrassi","name":"Joao Grassi","path":"/joaopgrassi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5938087?s=80&v=4"},"commit":{"message":"Metrics requirement levels moved to semconv repo","shortMessageHtmlLink":"Metrics requirement levels moved to semconv repo"}},{"before":"7145a5d2ae302070033c2088a2416812edb7f878","after":"012f594627801ca96b7c13ba55b568a6704dc05b","ref":"refs/heads/main","pushedAt":"2024-03-22T15:57:34.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"joaopgrassi","name":"Joao Grassi","path":"/joaopgrassi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5938087?s=80&v=4"},"commit":{"message":"Clarify that ReadableLogRecord and ReadWriteLogRecord can be represented using a single type (#3898)\n\nCurrently, the [Logs SDK specification](https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/logs/sdk.md#additional-logrecord-interfaces) says:\r\n\r\n> In addition to the [definition for LogRecord](https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/logs/data-model.md#log-and-event-record-definition), the following LogRecord-like interfaces are defined in the SDK: \r\n> ### ReadableLogRecord\r\n> [...]\r\n> Note: Typically this will be implemented with a new interface or (immutable) value type.\r\n> ### ReadWriteLogRecord\r\n\r\nDoes it means that the SDK has to literally define those types (abstractions)?\r\n\r\nThe problem is that each abstraction/type conversion can lead (depending on language) to performance overhead.\r\n\r\nE.g. for Go:\r\n- casting a type to an interface can lead to a heap allocation[^1][^2] which can noticeably affect the performance[^3]\r\n- converting to a different struct (value type) is also not free\r\n\r\nMoreover, having less abstractions reduces the API surface and makes the design simpler.\r\n\r\n[^1]: https://go101.org/optimizations/0.3-memory-allocations.html\r\n[^2]: https://github.com/open-telemetry/opentelemetry-go/blob/main/log/DESIGN.md#record-as-interface\r\n[^3]: https://tip.golang.org/doc/gc-guide#Eliminating_heap_allocations\r\n\r\nI believe that for Logs signal, performance is more important than API esthetics. Based on https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/logs/README.md, I think that the Logs SDK should be designed in a way to achieve to have high performance for a scenario when a OTLP exporter with a batch processor is used.\r\n\r\nIn my opinion, the `ReadableLogRecord`, `ReadWriteLogRecord` terms should only be used to describe what the functionalities accepting log records MUST be able to do with them. The key is that the log processor MUST be able to mutate the log record that is received in `OnEmit` while all other functionalities do not need mutate the record so they MAY  accept an immutable value.\r\n\r\nI noticed that [.NET SDK](https://github.com/open-telemetry/opentelemetry-dotnet/tree/main/src/OpenTelemetry/Logs) does not define anything like `ReadableLogRecord`\r\n\r\nMaybe it will help other languages as well in implementing the SDK.","shortMessageHtmlLink":"Clarify that ReadableLogRecord and ReadWriteLogRecord can be represen…"}},{"before":"bb3d0a0d14f41ccd3f78852316c77705e722edfa","after":"7145a5d2ae302070033c2088a2416812edb7f878","ref":"refs/heads/main","pushedAt":"2024-03-21T10:00:08.000Z","pushType":"push","commitsCount":64,"pusher":{"login":"joaopgrassi","name":"Joao Grassi","path":"/joaopgrassi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5938087?s=80&v=4"},"commit":{"message":"Clarify that exemplar reservoir default may change in a minor version (#3943)\n\nRelated to #3756. \r\n\r\nSee this\r\n[conversation](https://github.com/open-telemetry/opentelemetry-specification/pull/3870#discussion_r1483140692)\r\nfor context.\r\n\r\n---------\r\n\r\nCo-authored-by: Reiley Yang <reyang@microsoft.com>","shortMessageHtmlLink":"Clarify that exemplar reservoir default may change in a minor version (<a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"2188850080\" data-permission-text=\"Title is private\" data-url=\"https://github.com/open-telemetry/opentelemetry-specification/issues/3943\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/open-telemetry/opentelemetry-specification/pull/3943/hovercard\" href=\"https://github.com/open-telemetry/opentelemetry-specification/pull/3943\">…</a>"}},{"before":"a47b3b52bf776780e7b25ab65113a8d684897c33","after":null,"ref":"refs/heads/attributes-case-sensitive","pushedAt":"2023-12-20T14:55:54.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"arminru","name":"Armin Ruech","path":"/arminru","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/7052238?s=80&v=4"}},{"before":"56137d7cd17155c09e95a2534a2400428f961b28","after":"a47b3b52bf776780e7b25ab65113a8d684897c33","ref":"refs/heads/attributes-case-sensitive","pushedAt":"2023-12-18T19:31:11.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"carlosalberto","name":"Carlos Alberto Cortez","path":"/carlosalberto","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/260065?s=80&v=4"},"commit":{"message":"Merge branch 'main' into attributes-case-sensitive","shortMessageHtmlLink":"Merge branch 'main' into attributes-case-sensitive"}},{"before":"d2fea28a254d4b8361b306c285c926083b47ca9d","after":"56137d7cd17155c09e95a2534a2400428f961b28","ref":"refs/heads/attributes-case-sensitive","pushedAt":"2023-12-14T15:48:54.000Z","pushType":"push","commitsCount":3,"pusher":{"login":"arminru","name":"Armin Ruech","path":"/arminru","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/7052238?s=80&v=4"},"commit":{"message":"Merge branch 'main' into attributes-case-sensitive","shortMessageHtmlLink":"Merge branch 'main' into attributes-case-sensitive"}},{"before":"a1c79ce1b581e1b2edf7ae98bb9c193cb0b526fd","after":"d2fea28a254d4b8361b306c285c926083b47ca9d","ref":"refs/heads/attributes-case-sensitive","pushedAt":"2023-12-13T15:36:22.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"arminru","name":"Armin Ruech","path":"/arminru","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/7052238?s=80&v=4"},"commit":{"message":"Merge branch 'main' into attributes-case-sensitive","shortMessageHtmlLink":"Merge branch 'main' into attributes-case-sensitive"}},{"before":"016307bf417b0b55abe4552cd1f25aeaf67eae1e","after":"a1c79ce1b581e1b2edf7ae98bb9c193cb0b526fd","ref":"refs/heads/attributes-case-sensitive","pushedAt":"2023-12-11T14:48:58.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"arminru","name":"Armin Ruech","path":"/arminru","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/7052238?s=80&v=4"},"commit":{"message":"Merge branch 'main' into attributes-case-sensitive","shortMessageHtmlLink":"Merge branch 'main' into attributes-case-sensitive"}},{"before":"399e944283e00999bba0be1a423e9f9717b66e34","after":"016307bf417b0b55abe4552cd1f25aeaf67eae1e","ref":"refs/heads/attributes-case-sensitive","pushedAt":"2023-12-11T14:48:15.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"arminru","name":"Armin Ruech","path":"/arminru","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/7052238?s=80&v=4"},"commit":{"message":"Refine wording to not define the ultimate semantics but rather the handling within OTel","shortMessageHtmlLink":"Refine wording to not define the ultimate semantics but rather the ha…"}},{"before":"87db5357dddea3a4df525100d27a2c9fd4fa5f94","after":"399e944283e00999bba0be1a423e9f9717b66e34","ref":"refs/heads/attributes-case-sensitive","pushedAt":"2023-12-06T17:43:16.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"arminru","name":"Armin Ruech","path":"/arminru","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/7052238?s=80&v=4"},"commit":{"message":"Add changelog","shortMessageHtmlLink":"Add changelog"}},{"before":null,"after":"87db5357dddea3a4df525100d27a2c9fd4fa5f94","ref":"refs/heads/attributes-case-sensitive","pushedAt":"2023-12-06T17:36:23.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"arminru","name":"Armin Ruech","path":"/arminru","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/7052238?s=80&v=4"},"commit":{"message":"Clarify that attribute keys are case-sensitive","shortMessageHtmlLink":"Clarify that attribute keys are case-sensitive"}},{"before":"4229197f45f0da0642bc7c0127e84c8aa1439489","after":"bb3d0a0d14f41ccd3f78852316c77705e722edfa","ref":"refs/heads/main","pushedAt":"2023-12-06T17:25:41.000Z","pushType":"push","commitsCount":116,"pusher":{"login":"arminru","name":"Armin Ruech","path":"/arminru","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/7052238?s=80&v=4"},"commit":{"message":"Clarifications and \"flexibility\" fixes in Exemplar Specification (#3760)\n\nFixes #2205\r\nFixes #3674 \r\nFixes #3669\r\nPartially fixes #2421\r\n\r\n## Changes\r\n\r\n- Update example exemplar algorithm to account for initial reservoir\r\nfill\r\n- Update fixed-size defaults to account for memory contention /\r\noptimization in Java impl\r\n- Set a default for exponential histogram aggregation\r\n- Clarify that ExemplarFilter should be configured on MeterProvider\r\n- Make it clear that ONE reservoir is create PER timeseries datapoint\r\n(not one reservoir per view or metric name).\r\n- Allow flexibility in Reservoir `offer` definition based on feedback\r\nfrom Go impl.\r\n\r\n* Related issues #3756\r\n\r\n---------\r\n\r\nCo-authored-by: David Ashpole <dashpole@google.com>\r\nCo-authored-by: Joshua MacDonald <jmacd@users.noreply.github.com>","shortMessageHtmlLink":"Clarifications and \"flexibility\" fixes in Exemplar Specification (<a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"1988396675\" data-permission-text=\"Title is private\" data-url=\"https://github.com/open-telemetry/opentelemetry-specification/issues/3760\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/open-telemetry/opentelemetry-specification/pull/3760/hovercard\" href=\"https://github.com/open-telemetry/opentelemetry-specification/pull/3760\">ope…</a>"}},{"before":"fc6c93f73d90f1110c171e432a8e417f77123310","after":null,"ref":"refs/heads/eager-bsp","pushedAt":"2023-06-15T20:09:53.149Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"dyladan","name":"Daniel Dyla","path":"/dyladan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1612643?s=80&v=4"}},{"before":null,"after":"fc6c93f73d90f1110c171e432a8e417f77123310","ref":"refs/heads/eager-bsp","pushedAt":"2023-06-15T20:09:41.287Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"dyladan","name":"Daniel Dyla","path":"/dyladan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1612643?s=80&v=4"},"commit":{"message":"Merge branch 'main' into eager-bsp","shortMessageHtmlLink":"Merge branch 'main' into eager-bsp"}},{"before":"fc6c93f73d90f1110c171e432a8e417f77123310","after":null,"ref":"refs/heads/eager-bsp","pushedAt":"2023-06-15T20:09:35.897Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"dyladan","name":"Daniel Dyla","path":"/dyladan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1612643?s=80&v=4"}},{"before":"b5afddaa815cf1eac7e631def9597277821522b5","after":null,"ref":"refs/heads/update-outdated-hostname","pushedAt":"2023-05-22T18:24:59.974Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"arminru","name":"Armin Ruech","path":"/arminru","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/7052238?s=80&v=4"}},{"before":null,"after":"b5afddaa815cf1eac7e631def9597277821522b5","ref":"refs/heads/update-outdated-hostname","pushedAt":"2023-05-22T09:57:19.493Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"joaopgrassi","name":"Joao Grassi","path":"/joaopgrassi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5938087?s=80&v=4"},"commit":{"message":"Fix outdated host.hostname attribute in log data model","shortMessageHtmlLink":"Fix outdated host.hostname attribute in log data model"}},{"before":"90007a192ae8b06d40ee8c15f56256d69a330321","after":"4229197f45f0da0642bc7c0127e84c8aa1439489","ref":"refs/heads/main","pushedAt":"2023-05-22T09:52:07.898Z","pushType":"push","commitsCount":35,"pusher":{"login":"joaopgrassi","name":"Joao Grassi","path":"/joaopgrassi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5938087?s=80&v=4"},"commit":{"message":"Release 1.21.0 (#3474)\n\nMay Release.","shortMessageHtmlLink":"Release 1.21.0 (<a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"1697662281\" data-permission-text=\"Title is private\" data-url=\"https://github.com/open-telemetry/opentelemetry-specification/issues/3474\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/open-telemetry/opentelemetry-specification/pull/3474/hovercard\" href=\"https://github.com/open-telemetry/opentelemetry-specification/pull/3474\">open-telemetry#3474</a>)"}},{"before":"a45eeedbe99284e66f2032204f974519230090d0","after":null,"ref":"refs/heads/remove-dup-text","pushedAt":"2023-04-18T15:31:44.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"joaopgrassi","name":"Joao Grassi","path":"/joaopgrassi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5938087?s=80&v=4"}},{"before":null,"after":"a45eeedbe99284e66f2032204f974519230090d0","ref":"refs/heads/remove-dup-text","pushedAt":"2023-04-18T14:58:17.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"joaopgrassi","name":"Joao Grassi","path":"/joaopgrassi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5938087?s=80&v=4"},"commit":{"message":"Remove duplicated text from appender section","shortMessageHtmlLink":"Remove duplicated text from appender section"}},{"before":"639864e91d11f73279305c8113cc90f181709e3d","after":"90007a192ae8b06d40ee8c15f56256d69a330321","ref":"refs/heads/main","pushedAt":"2023-04-18T11:49:59.572Z","pushType":"push","commitsCount":86,"pusher":{"login":"joaopgrassi","name":"Joao Grassi","path":"/joaopgrassi","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/5938087?s=80&v=4"},"commit":{"message":"Change http.server.duration and http.client.duration units to seconds (#3390)","shortMessageHtmlLink":"Change http.server.duration and http.client.duration units to seconds (<a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"1665632929\" data-permission-text=\"Title is private\" data-url=\"https://github.com/open-telemetry/opentelemetry-specification/issues/3390\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/open-telemetry/opentelemetry-specification/pull/3390/hovercard\" href=\"https://github.com/open-telemetry/opentelemetry-specification/pull/3390\">…</a>"}},{"before":"96870b0d0e6240e24a1a95b889ba99b43c91f6dd","after":null,"ref":"refs/heads/feature/faas-cloud-id-ref","pushedAt":"2023-04-11T13:54:50.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"Oberon00","name":"Christian Neumüller","path":"/Oberon00","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/849039?s=80&v=4"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAAESUrluwA","startCursor":null,"endCursor":null}},"title":"Activity · dynatrace-oss-contrib/opentelemetry-specification"}