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

Cannot register grain activations using the Azure table-based directory #8148

Closed
NSTA2 opened this issue Nov 15, 2022 · 3 comments · Fixed by #8151
Closed

Cannot register grain activations using the Azure table-based directory #8148

NSTA2 opened this issue Nov 15, 2022 · 3 comments · Fixed by #8151
Assignees

Comments

@NSTA2
Copy link

NSTA2 commented Nov 15, 2022

Porting a project to Orleans 7 caused an issue for me:

A grain which has both transactional state and uses a persisted grain directory to ensure only one instance of the grain will ever exist causes this error (which was not the case using 3.6.5)

        Orleans.Runtime.OrleansMessageRejectionException: 
        'Forwarding failed: tried to forward message Request [S127.0.0.1:11111:27517279 sys.client/hosted-127.0.0.1:11111@27517279]->[S127.0.0.1:11111:27517279 transactiongrainwithdirectory/B]
        GrainDirectoryWithTransactionIssueReproduction.ITransactionGrainWithDirectoryGrainDirectoryWithTransactionIssueReproduction.ITransactionGrainWithDirectory.SayHi() 
        #28[ForwardCount=2] for 2 times after "Failed to register activation in grain directory." to invalid activation. Rejecting now. '

Removing the GrainDirectory attribute on the affected grain resolved the issue for me.

Here is a reproduction of the issue

@ghost ghost added the Needs: triage 🔍 label Nov 15, 2022
@benjaminpetit
Copy link
Member

It seems that the Azure table-based directory is broken in 7.0

@NSTA2
Copy link
Author

NSTA2 commented Nov 15, 2022

Ah, so nothing to do with transactions? (should I correct the title?)

@benjaminpetit benjaminpetit changed the title Persisted grain directory + transactional state = Exception Cannot register grain activations using the Azure table-based directory Nov 15, 2022
@benjaminpetit
Copy link
Member

Done :) Thanks for catching the issue!

@ghost ghost added the Status: Fixed label Nov 15, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Dec 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants