You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2022/02/23 13:30:09 UTC

[GitHub] [geode-native] gaussianrecurrence opened a new pull request #930: GEODE-10044: Allow FunctionAttributes update

gaussianrecurrence opened a new pull request #930:
URL: https://github.com/apache/geode-native/pull/930


    - Moved FunctionExecution and OnRegionFunctionExecution into a separate
      file in order to improve readability and code maintenance.
    - Moved ChunkedFunctionExecutionResponse into a separate file in order
      to improve readability and code maintenance.
    - Introduced FunctionAttributes class to improve code readability.
    - Modified code to introduce the usage of FunctionAttributes.
    - Modified TcrMessageTest UT to work with the inclusion of
      FunctionAttributes.
    - Implemented a mechanism to cleanup function attributes.
    - A new exception FunctionAttributesMismatchException is added to be
      thrown whenever the server detects a mismatch for the function
      attributes.
    - Now whenever FunctionAttributesMismatchException is catched while
      executing a function, its function attributes are cleaned up, so the
      client will request an update the next execution.
    - Added a new IT to verify that chaging a function attributes on the
      fly is possible.


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [geode-native] lgtm-com[bot] commented on pull request #930: GEODE-10044: Allow FunctionAttributes update

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #930:
URL: https://github.com/apache/geode-native/pull/930#issuecomment-1048817585


   This pull request **introduces 1 alert** when merging 2fad8a96ad07192f0fdf5efd539ec37ee0b23be2 into 9aa78f4c7eae06618f0598fee60b64f9ea127d16 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode-native/rev/pr-38ba6bc4713db77e3a735760b5dacd2b44133972)
   
   **new alerts:**
   
   * 1 for Too few arguments to formatting function


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [geode-native] gaussianrecurrence commented on pull request #930: GEODE-10044: Allow FunctionAttributes update

Posted by GitBox <gi...@apache.org>.
gaussianrecurrence commented on pull request #930:
URL: https://github.com/apache/geode-native/pull/930#issuecomment-1048786258


   The rationale behind this change is allowing the server function upgrade scenario, which was previously discussed in here: [https://www.mail-archive.com/dev@geode.apache.org/msg26144.html](https://www.mail-archive.com/dev@geode.apache.org/msg26144.html)


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org