chore: remove unused code (#6768)

This code is not used anymore and has no effect. Reintroducing it
does not seem to be ideal. It is likely best to always name the
services uniquely.

Fixes: #6766

Author: BridgeAR

docs/test.ts

@@ -262,7 +262,6 @@ const elasticsearchOptions: plugins.elasticsearch = {
 
 const awsSdkOptions: plugins.aws_sdk = {
   service: 'test',
-  splitByAwsService: false,
   batchPropagationEnabled: false,
   hooks: {
     request: (span?: Span, response?) => {},

index.d.ts

@@ -1698,12 +1698,6 @@ declare namespace tracer {
      * [aws-sdk](https://github.com/aws/aws-sdk-js) module.
      */
     interface aws_sdk extends Instrumentation {
-      /**
-       * Whether to add a suffix to the service name so that each AWS service has its own service name.
-       * @default true
-       */
-      splitByAwsService?: boolean;
-
       /**
        * Whether to inject all messages during batch AWS SQS, Kinesis, and SNS send operations. Normal
        * behavior is to inject the first message in batch send operations.

packages/datadog-plugin-aws-sdk/src/base.js

@@ -282,7 +282,6 @@ function normalizeConfig (config, serviceIdentifier) {
   return {
     ...config,
     ...specificConfig,
-    splitByAwsService: config.splitByAwsService !== false,
     batchPropagationEnabled,
     hooks
   }

packages/datadog-plugin-aws-sdk/test/aws-sdk.spec.js

@@ -254,7 +254,6 @@ describe('Plugin', () => {
         before(() => {
           return agent.load(['aws-sdk', 'http'], [{
             service: 'test',
-            splitByAwsService: false,
             hooks: {
               request (span, response) {
                 span.setTag('hook.operation', response.request.operation)