[ 
https://issues.apache.org/jira/browse/GEODE-3192?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16096984#comment-16096984
 ] 

ASF GitHub Bot commented on GEODE-3192:
---------------------------------------

Github user PurelyApplied commented on the issue:

    https://github.com/apache/geode/pull/643
  
    If you are using IntelliJ as your IDE, please remember to update your style 
file to that located in `geode/etc/intellij-java-modified-google-style.xml` to 
be consistent with Geode's established style guide.  This file was updated June 
13 with commit `a561bd12` to be consistent with documented expectations.  If 
you are using Eclipse, please import both style guide files, also located in 
`geode/etc/`.
    
    After having done so, please optimize the imports to the correct order in 
each of the `.java` files touched. 



> PutAll on new protocol informs client which keys couldn't be inserted
> ---------------------------------------------------------------------
>
>                 Key: GEODE-3192
>                 URL: https://issues.apache.org/jira/browse/GEODE-3192
>             Project: Geode
>          Issue Type: Improvement
>          Components: client/server
>            Reporter: Alexander Murmann
>
> When a putAll request fails to insert a given key/value pair the 
> corresponding response will contain information about which keys/values 
> failed to be inserted.
> *Note:* We left this out of GEODE-2997 which initially introduces putAll 
> because we want to get something to test with early.
> *Implementation Note:* The new protocol uses Region#putAll which returns 
> void, but the old protocol uses basicBridgePutAll, which we don't understand 
> but which seems to give more information. Having a look at the old protocol's 
> putAll command could be useful.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to