Addressing feedback

Author: aws-rizi

README.md

@@ -1,7 +1,9 @@
 ## Large Payload Offloading Java Common Library For AWS
 
 The **Large Payload Offloading Java Common Library For AWS** enables you to manage payloads with Amazon S3. 
-This is especially useful for storing and retrieving payloads with size greater than the current SQS/SNS limit of 256 KB, up to a maximum of 2 GB. Specifically, you can use this library to:
+For example, this is useful for storing and retrieving payloads with size greater than the current SQS/SNS limit of 256 KB, up to a maximum of 2 GB. Specifically.
+ 
+You can use this library to:
 
 * Specify whether payloads are always stored in Amazon S3 or only when a payload's size exceeds 256 KB.
 

pom.xml

@@ -40,11 +40,6 @@
     </properties>
 
     <dependencies>
-        <dependency>
-            <groupId>com.amazonaws</groupId>
-            <artifactId>aws-java-sdk-sqs</artifactId>
-            <version>${aws-java-sdk.version}</version>
-        </dependency>
         <dependency>
             <groupId>com.amazonaws</groupId>
             <artifactId>aws-java-sdk-s3</artifactId>

src/main/java/com/amazonaws/largepayloadoffloading/Constants.java

@@ -0,0 +1,9 @@
+package com.amazonaws.largepayloadoffloading;
+
+public class Constants {
+    /**
+	 * RESERVED_ATTRIBUTE_NAME is shared between SQSExtendedClient and SNSExtendedClient as an attribute key.
+	 * This attribute indicates that the message is a pointer to the stored payload and its value shows size of actual payload.
+     */
+    public static final String RESERVED_ATTRIBUTE_NAME = "AWSOffloadedPayloadSize";
+}

src/main/java/com/amazonaws/largepayloadoffloading/CountingOutputStream.java

@@ -9,24 +9,24 @@
  * strings.
  */
 class CountingOutputStream extends OutputStream {
-	private long totalSize;
+    private long totalSize;
 
-	@Override
-	public void write(int b) {
-		++totalSize;
-	}
+    @Override
+    public void write(int b) {
+        ++totalSize;
+    }
 
-	@Override
-	public void write(byte[] b) {
-		totalSize += b.length;
-	}
+    @Override
+    public void write(byte[] b) {
+        totalSize += b.length;
+    }
 
-	@Override
-	public void write(byte[] b, int offset, int len) {
-		totalSize += len;
-	}
+    @Override
+    public void write(byte[] b, int offset, int len) {
+        totalSize += len;
+    }
 
-	public long getTotalSize() {
-		return totalSize;
-	}
+    public long getTotalSize() {
+        return totalSize;
+    }
 }

src/main/java/com/amazonaws/largepayloadoffloading/ExtendedClientConfiguration.java

@@ -13,26 +13,25 @@
  * object.
  */
 class JsonDataConverter {
+    protected final ObjectMapper objectMapper;
 
-	protected final ObjectMapper objectMapper;
+    public JsonDataConverter() {
+        this(new ObjectMapper());
+        objectMapper.configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false);
+        objectMapper.configure(SerializationFeature.FAIL_ON_EMPTY_BEANS, false);
+        objectMapper.enableDefaultTyping(DefaultTyping.NON_FINAL);
+    }
 
-	public JsonDataConverter() {
-		this(new ObjectMapper());
-		objectMapper.configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false);
-		objectMapper.configure(SerializationFeature.FAIL_ON_EMPTY_BEANS, false);
-		objectMapper.enableDefaultTyping(DefaultTyping.NON_FINAL);
-	}
+    public JsonDataConverter(ObjectMapper objectMapper) {
+        this.objectMapper = objectMapper;
+    }
 
-	public JsonDataConverter(ObjectMapper objectMapper) {
-		this.objectMapper = objectMapper;
-	}
+    public String serializeToJson(Object obj) throws JsonProcessingException {
+        ObjectWriter objectWriter = objectMapper.writer();
+        return objectWriter.writeValueAsString(obj);
+    }
 
-	public String serializeToJson(Object obj) throws JsonProcessingException {
-		ObjectWriter objectWriter = objectMapper.writer();
-		return objectWriter.writeValueAsString(obj);
-	}
-
-	public <T> T deserializeFromJson(String jsonText, Class<T> objectType) throws Exception {
-		return objectMapper.readValue(jsonText, objectType);
-	}
+    public <T> T deserializeFromJson(String jsonText, Class<T> objectType) throws Exception {
+        return objectMapper.readValue(jsonText, objectType);
+    }
 }