Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GEODE-10400: Fix metadata completion detection #982

Closed

Conversation

gaussianrecurrence
Copy link
Contributor

  • Whenever execution server functions with isHA=false, hasResult=true
    and optimizeForWrite=true, if the metadata is incomplete a request of
    type EXECUTE_REGION_FUNCTION_SINGLE_HOP is sent with an invalid
    bucket set partition. This causes an
    InternalFunctionInvocationTargetException exception
  • In order to solve this issue, now, whenever calling
    groupByServerToBuckets, if there isn't a valid location for one of
    the buckets, it returns nullptr, triggering the fallback mechanism
    which would be sending EXECUTE_REGION_FUNCTION to one of the nodes
    instead.

 - Whenever execution server functions with isHA=false, hasResult=true
   and optimizeForWrite=true, if the metadata is incomplete a request of
   type EXECUTE_REGION_FUNCTION_SINGLE_HOP is sent with an invalid
   bucket set partition. This causes an
   InternalFunctionInvocationTargetException exception
 - In order to solve this issue, now, whenever calling
   groupByServerToBuckets, if there isn't a valid location for one of
   the buckets, it returns nullptr, triggering the fallback mechanism
   which would be sending EXECUTE_REGION_FUNCTION to one of the nodes
   instead.
@gaussianrecurrence
Copy link
Contributor Author

I haven't had the time to work on this for a long time or I think I'll have the opportunity to in the near future, hence I am closing this PR, if anyone wants to tackle this issue can take it up from its latest state.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant