Add getters/accessors for readable fields in ReadWriteLogRecord.

docs/apidiffs/current_vs_latest/opentelemetry-sdk-logs.txt

@@ -1,2 +1,21 @@
 Comparing source compatibility of opentelemetry-sdk-logs-1.46.0-SNAPSHOT.jar against opentelemetry-sdk-logs-1.45.0.jar
-No changes.
+***  MODIFIED INTERFACE: PUBLIC ABSTRACT io.opentelemetry.sdk.logs.ReadWriteLogRecord  (not serializable)
+	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
+	+++  NEW INTERFACE: io.opentelemetry.sdk.logs.data.LogRecordData
+	+++  NEW METHOD: PUBLIC(+) java.lang.Object getAttribute(io.opentelemetry.api.common.AttributeKey<T>)
+		+++  NEW ANNOTATION: javax.annotation.Nullable
+		GENERIC TEMPLATES: +++ T:java.lang.Object
+	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.api.common.Attributes getAttributes()
+	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.sdk.logs.data.Body getBody()
+		+++  NEW ANNOTATION: java.lang.Deprecated
+	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.api.common.Value<?> getBodyValue()
+		+++  NEW ANNOTATION: javax.annotation.Nullable
+	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.sdk.common.InstrumentationScopeInfo getInstrumentationScopeInfo()
+	+++  NEW METHOD: PUBLIC(+) long getObservedTimestampEpochNanos()
+	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.sdk.resources.Resource getResource()
+	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.api.logs.Severity getSeverity()
+	+++  NEW METHOD: PUBLIC(+) java.lang.String getSeverityText()
+		+++  NEW ANNOTATION: javax.annotation.Nullable
+	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.api.trace.SpanContext getSpanContext()
+	+++  NEW METHOD: PUBLIC(+) long getTimestampEpochNanos()
+	+++  NEW METHOD: PUBLIC(+) int getTotalAttributeCount()

sdk/logs/src/main/java/io/opentelemetry/sdk/logs/ReadWriteLogRecord.java

@@ -7,14 +7,20 @@
 
 import io.opentelemetry.api.common.AttributeKey;
 import io.opentelemetry.api.common.Attributes;
+import io.opentelemetry.api.common.Value;
+import io.opentelemetry.api.logs.Severity;
+import io.opentelemetry.api.trace.SpanContext;
+import io.opentelemetry.sdk.common.InstrumentationScopeInfo;
 import io.opentelemetry.sdk.logs.data.LogRecordData;
+import io.opentelemetry.sdk.resources.Resource;
+import javax.annotation.Nullable;
 
 /**
  * A log record that can be read from and written to.
  *
  * @since 1.27.0
  */
-public interface ReadWriteLogRecord {
+public interface ReadWriteLogRecord extends LogRecordData {
 
   /**
    * Sets an attribute on the log record. If the log record previously contained a mapping for the
@@ -47,7 +53,87 @@
   /** Return an immutable {@link LogRecordData} instance representing this log record. */
   LogRecordData toLogRecordData();
 
-  // TODO: add additional log record accessors. Currently, all fields can be accessed indirectly via
-  // #toLogRecordData() at the expense of additional allocations.
+  /**
+   * Returns the value of a given attribute if it exists. This is the equivalent of calling
+   * getAttributes().get(key)
+   */
+  @Nullable
+  default <T> T getAttribute(AttributeKey<T> key) {
+    return toLogRecordData().getAttributes().get(key);
+  }
+
+  /** Returns the resource of this log record. */
+  @Override
+  default Resource getResource() {
+    return toLogRecordData().getResource();
+  }
+
+  /** Returns the instrumentation scope that generated this log. */
+  @Override
+  default InstrumentationScopeInfo getInstrumentationScopeInfo() {
+    return toLogRecordData().getInstrumentationScopeInfo();
+  }
 
+  /** Returns the timestamp at which the log record occurred, in epoch nanos. */
+  @Override
+  default long getTimestampEpochNanos() {
+    return toLogRecordData().getTimestampEpochNanos();
+  }
+
+  /** Returns the timestamp at which the log record was observed, in epoch nanos. */
+  @Override
+  default long getObservedTimestampEpochNanos() {
+    return toLogRecordData().getTimestampEpochNanos();
+  }
+
+  /** Return the span context for this log, or {@link SpanContext#getInvalid()} if unset. */
+  @Override
+  default SpanContext getSpanContext() {
+    return toLogRecordData().getSpanContext();
+  }
+
+  /** Returns the severity for this log, or {@link Severity#UNDEFINED_SEVERITY_NUMBER} if unset. */
+  @Override
+  default Severity getSeverity() {
+    return toLogRecordData().getSeverity();
+  }
+
+  /** Returns the severity text for this log, or null if unset. */
+  @Nullable
+  @Override
+  default String getSeverityText() {
+    return toLogRecordData().getSeverityText();
+  }
+
+  @Deprecated
+  @Override
+  @SuppressWarnings("deprecation") // Implementation of deprecated method
+  default io.opentelemetry.sdk.logs.data.Body getBody() {
+    return toLogRecordData().getBody();
+  }
+
+  /** Returns the {@link Value} representation of the log body, of null if unset. */
+  @Nullable
+  @Override
+  default Value<?> getBodyValue() {
+    return toLogRecordData().getBodyValue();
+  }
+
+  /** Returns the attributes for this log, or {@link Attributes#empty()} if unset. */
+  @Override
+  default Attributes getAttributes() {
+    return toLogRecordData().getAttributes();
+  }
+
+  /**
+   * Returns the total number of attributes that were recorded on this log.
+   *
+   * <p>This number may be larger than the number of attributes that are attached to this log, if
+   * the total number recorded was greater than the configured maximum value. See {@link
+   * LogLimits#getMaxNumberOfAttributes()}.
+   */
+  @Override
+  default int getTotalAttributeCount() {
+    return toLogRecordData().getTotalAttributeCount();
+  }
 }

sdk/logs/src/main/java/io/opentelemetry/sdk/logs/SdkReadWriteLogRecord.java

@@ -125,4 +125,72 @@
           attributes == null ? 0 : attributes.getTotalAddedValues());
     }
   }
+
+  @Override
+  public Resource getResource() {
+    return resource;
+  }
+
+  @Override
+  public InstrumentationScopeInfo getInstrumentationScopeInfo() {
+    return instrumentationScopeInfo;
+  }
+
+  @Override
+  public long getTimestampEpochNanos() {
+    return timestampEpochNanos;
+  }
+
+  @Override
+  public long getObservedTimestampEpochNanos() {
+    return observedTimestampEpochNanos;
+  }
+
+  @Override
+  public SpanContext getSpanContext() {
+    return spanContext;
+  }
+
+  @Override
+  public Severity getSeverity() {
+    return severity;
+  }
+
+  @Nullable
+  @Override
+  public String getSeverityText() {
+    return severityText;
+  }
+
+  @Nullable
+  @Override
+  public Value<?> getBodyValue() {
+    return body;
+  }
+
+  @Override
+  public Attributes getAttributes() {
+    return getImmutableAttributes();
+  }
+
+  @Nullable
+  @Override
+  public <T> T getAttribute(AttributeKey<T> key) {
+    synchronized (lock) {
+      if (attributes == null || attributes.isEmpty()) {
+        return null;
+      }
+      return attributes.get(key);
+    }
+  }
+
+  @Override
+  public int getTotalAttributeCount() {
+    synchronized (lock) {
+      if (attributes == null || attributes.isEmpty()) {
+        return 0;
+      }
+      return attributes.size();
+    }
+  }
 }