Delete DbClientCommonAttributesGetter and consolidate into DbClientAttributesGetter

- Removed DbClientCommonAttributesGetter interface entirely
- Added deprecated getDbSystem() and getResponseStatus() methods to DbClientAttributesGetter
  with default implementations that delegate to the new method names
- This provides a one-release deprecation cycle since these methods weren't deprecated
  in the previous release
- Also carried over deprecated getUser() and getConnectionString() methods
- All functionality now consolidated in DbClientAttributesGetter

Author: trask

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

@@ -25,6 +25,14 @@ public interface DbClientAttributesGetter<REQUEST, RESPONSE>
 
   String getDbSystemName(REQUEST request);
 
+  /**
+   * @deprecated Use {@link #getDbSystemName(REQUEST)} instead.
+   */
+  @Deprecated
+  default String getDbSystem(REQUEST request) {
+    return getDbSystemName(request);
+  }
+
   @Nullable
   String getDbNamespace(REQUEST request);
 
@@ -33,6 +41,27 @@ default String getResponseStatusCode(@Nullable RESPONSE response, @Nullable Thro
     return null;
   }
 
+  /**
+   * @deprecated Use {@link #getResponseStatusCode(RESPONSE, Throwable)} instead.
+   */
+  @Deprecated
+  @Nullable
+  default String getResponseStatus(@Nullable RESPONSE response, @Nullable Throwable error) {
+    return getResponseStatusCode(response, error);
+  }
+
+  @Deprecated
+  @Nullable
+  default String getUser(REQUEST request) {
+    return null;
+  }
+
+  @Deprecated
+  @Nullable
+  default String getConnectionString(REQUEST request) {
+    return null;
+  }
+
   /**
    * @deprecated Use {@link #getDbQueryText(REQUEST)} instead.
    */