Unhook DbClientAttributesGetter from DbClientCommonAttributesGetter

- Removed extends DbClientCommonAttributesGetter from DbClientAttributesGetter
- Copied non-deprecated methods (getDbSystemName, getDbNamespace, getResponseStatusCode)
  directly into DbClientAttributesGetter
- This makes DbClientCommonAttributesGetter fully deprecated and independent
- Maintains backward compatibility while moving towards removal of the deprecated interface

Author: trask

instrumentation-api-incubator/src/main/java/io/opentelemetry/instrumentation/api/incubator/semconv/db/DbClientAttributesGetter.java

@@ -20,11 +20,18 @@
  * from the attribute methods, but implement as many as possible for best compliance with the
  * OpenTelemetry specification.
  */
-@SuppressWarnings("deprecation") // until DbClientCommonAttributesGetter is removed
 public interface DbClientAttributesGetter<REQUEST, RESPONSE>
-    extends DbClientCommonAttributesGetter<REQUEST, RESPONSE>,
-        NetworkAttributesGetter<REQUEST, RESPONSE>,
-        ServerAttributesGetter<REQUEST> {
+    extends NetworkAttributesGetter<REQUEST, RESPONSE>, ServerAttributesGetter<REQUEST> {
+
+  String getDbSystemName(REQUEST request);
+
+  @Nullable
+  String getDbNamespace(REQUEST request);
+
+  @Nullable
+  default String getResponseStatusCode(@Nullable RESPONSE response, @Nullable Throwable error) {
+    return null;
+  }
 
   /**
    * @deprecated Use {@link #getDbQueryText(REQUEST)} instead.