AWS versions bump

Author: jtduffy

CONTRIBUTING.md

@@ -37,17 +37,8 @@ Before we can accept a pull request, you must sign our [Contributor Licensing Ag
 Minimally, the [test suite](#testing-guidelines) must pass for us to accept a PR. Ideally, we would love it if you also added appropriate tests if you're implementing a feature!
 
 ## Coding Style Guidelines
-Our code base is formatted according to the [Google Java Style Guide](https://google.github.io/styleguide/javaguide.html).
-
-The project is configured to use the [google-java-format-gradle-plugin](https://github.com/sherter/google-java-format-gradle-plugin) which can be utilized as follows:  
-
-* Execute the task `googleJavaFormat` to format all `*.java` files in the project:
-
-`./gradlew goJF`
-
-* Execute the task `verifyGoogleJavaFormat` to verify that all `*.java` files are formatted properly:
-
-`./gradlew verGJF`
+- Use the style provided in `dev-tools/code-style/java-agent-code-style.xml` in the project.
+- We encourage you to reduce tech debt you might find in the area. Leave the code better than you found it.
 
 ## Testing Guidelines
 The AWS Lambda OpenTracing Java SDK comes with tests in `src/test` that can be run with `./gradlew test`.

build.gradle

@@ -7,9 +7,6 @@ buildscript {
         }
 
     }
-    dependencies {
-        classpath "gradle.plugin.com.github.sherter.google-java-format:google-java-format-gradle-plugin:0.9"
-    }
 }
 
 plugins {
@@ -25,8 +22,6 @@ java {
     withJavadocJar()
 }
 
-apply plugin: 'com.github.sherter.google-java-format'
-
 repositories {
     mavenCentral()
     maven {
@@ -35,12 +30,22 @@ repositories {
 }
 
 dependencies {
-    implementation 'com.amazonaws:aws-lambda-java-core:1.1.0'
+    //implementation 'software.amazon.awssdk:bom:2.31.43'
+
+    implementation 'software.amazon.awssdk:s3:2.31.43'
+    implementation 'software.amazon.awssdk:s3-event-notifications:2.31.43'
+    //implementation 'software.amazon.awssdk:kinesis:2.31.43'
+    //implementation 'software.amazon.awssdk:dynamodb:2.31.43'
+
+    implementation 'com.amazonaws:aws-lambda-java-core:1.2.3'
     // 2.2.7 is earliest version that has all needed event sources
-    implementation 'com.amazonaws:aws-lambda-java-events:2.2.7'
-    implementation 'com.amazonaws:aws-java-sdk-s3:1.12.771'
-    implementation 'com.amazonaws:aws-java-sdk-kinesis:1.11.163'
-    implementation 'com.amazonaws:aws-java-sdk-dynamodb:1.11.163'
+    implementation 'com.amazonaws:aws-lambda-java-events:3.15.0'
+
+
+    //implementation 'com.amazonaws:aws-java-sdk-s3:1.12.771'
+    //implementation 'com.amazonaws:aws-java-sdk-kinesis:1.11.163'
+    //implementation 'com.amazonaws:aws-java-sdk-dynamodb:1.11.163'
+
     implementation('io.opentracing:opentracing-api:0.33.0')
     implementation('io.opentracing:opentracing-util:0.33.0')
     implementation('io.opentracing:opentracing-noop:0.33.0')

src/main/java/com/newrelic/opentracing/aws/EnhancedSpanBuilder.java

@@ -10,66 +10,72 @@
 import io.opentracing.Tracer;
 
 public class EnhancedSpanBuilder {
-  private final Tracer.SpanBuilder spanBuilder;
+    private final Tracer.SpanBuilder spanBuilder;
 
-  public static EnhancedSpanBuilder basedOn(Tracer tracer, String operationName) {
-    return new EnhancedSpanBuilder(tracer.buildSpan(operationName));
-  }
+    public static EnhancedSpanBuilder basedOn(Tracer tracer, String operationName) {
+        return new EnhancedSpanBuilder(tracer.buildSpan(operationName));
+    }
 
-  private EnhancedSpanBuilder(Tracer.SpanBuilder innerSpanBuilder) {
-    spanBuilder = innerSpanBuilder;
-  }
+    private EnhancedSpanBuilder(Tracer.SpanBuilder innerSpanBuilder) {
+        spanBuilder = innerSpanBuilder;
+    }
 
-  public EnhancedSpanBuilder asChildOf(SpanContext spanContext) {
-    this.spanBuilder.asChildOf(spanContext);
-    return this;
-  }
+    public EnhancedSpanBuilder asChildOf(SpanContext spanContext) {
+        this.spanBuilder.asChildOf(spanContext);
+        return this;
+    }
 
-  /** Same as {@link Span#setTag(String, String)}, but for the span to be built. */
-  EnhancedSpanBuilder withTag(String key, String value) {
-    this.spanBuilder.withTag(key, value);
-    return this;
-  }
+    /**
+     * Same as {@link Span#setTag(String, String)}, but for the span to be built.
+     */
+    EnhancedSpanBuilder withTag(String key, String value) {
+        this.spanBuilder.withTag(key, value);
+        return this;
+    }
 
-  /** Same as {@link Span#setTag(String, boolean)}, but for the span to be built. */
-  EnhancedSpanBuilder withTag(String key, boolean value) {
-    this.spanBuilder.withTag(key, value);
-    return this;
-  }
+    /**
+     * Same as {@link Span#setTag(String, boolean)}, but for the span to be built.
+     */
+    EnhancedSpanBuilder withTag(String key, boolean value) {
+        this.spanBuilder.withTag(key, value);
+        return this;
+    }
 
-  /** Same as {@link Span#setTag(String, Number)}, but for the span to be built. */
-  EnhancedSpanBuilder withTag(String key, Number value) {
-    this.spanBuilder.withTag(key, value);
-    return this;
-  }
+    /**
+     * Same as {@link Span#setTag(String, Number)}, but for the span to be built.
+     */
+    EnhancedSpanBuilder withTag(String key, Number value) {
+        this.spanBuilder.withTag(key, value);
+        return this;
+    }
 
-  /**
-   * A shorthand for withTag("key", "value").
-   *
-   * <p>If parent==null, this is a noop.
-   */
-  EnhancedSpanBuilder optionallyWithTag(String key, String value) {
-    if (value != null) {
-      this.spanBuilder.withTag(key, value);
+    /**
+     * A shorthand for withTag("key", "value").
+     *
+     * <p>If parent==null, this is a noop.
+     */
+    EnhancedSpanBuilder optionallyWithTag(String key, String value) {
+        if (value != null) {
+            this.spanBuilder.withTag(key, value);
+        }
+        return this;
     }
-    return this;
-  }
 
-  EnhancedSpanBuilder optionallyWithTag(String key, boolean value) {
-    if (value) {
-      this.spanBuilder.withTag(key, true);
+    EnhancedSpanBuilder optionallyWithTag(String key, boolean value) {
+        if (value) {
+            this.spanBuilder.withTag(key, true);
+        }
+        return this;
     }
-    return this;
-  }
 
-  EnhancedSpanBuilder optionallyWithTag(String key, Number value) {
-    if (value != null) {
-      this.spanBuilder.withTag(key, value);
+    EnhancedSpanBuilder optionallyWithTag(String key, Number value) {
+        if (value != null) {
+            this.spanBuilder.withTag(key, value);
+        }
+        return this;
     }
-    return this;
-  }
 
-  public Span start() {
-    return this.spanBuilder.start();
-  }
+    public Span start() {
+        return this.spanBuilder.start();
+    }
 }