dscalzi opened a new pull request #5748:
URL: https://github.com/apache/camel/pull/5748


   Switch to use the existing parent class's method to determine the exclusive 
start key, only set it once in the builder.
   
   ---
   
   Side note, the following line will set the LAST_EVALUATED_KEY to an 
SdkAutoConstructMap when no last evaluated key is present.
   
   ```java
   tmp.put(Ddb2Constants.LAST_EVALUATED_KEY, result.lastEvaluatedKey());
   ```
   
   I'm not sure if this is intentional, or the component should check for this 
and return null. Currently, dependent applications have to reimplement this 
logic available in the sdk.
   
   ```java
   public final boolean hasLastEvaluatedKey() {
       return lastEvaluatedKey != null && !(lastEvaluatedKey instanceof 
SdkAutoConstructMap);
   }
   ```
   
   If an always-null if not present is desired, I could put up another pull 
request.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to