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

Conversation handle 00000000-00000 etc. not found #75

Closed
OriginalDREiD opened this issue Jun 13, 2018 · 13 comments
Closed

Conversation handle 00000000-00000 etc. not found #75

OriginalDREiD opened this issue Jun 13, 2018 · 13 comments

Comments

@OriginalDREiD
Copy link

I have been using SQLDepency for my application for a while and came across yours which looks great for my requirements.

I got the code up and running but one I've started the tabledependency, edits to the table results in the conversation handle 00000000-0000-0000-0000-000000000000 is not found.

If I stop the tabledependency, edits work fine.

Thanks.

@elclon
Copy link

elclon commented Jun 16, 2018

Hi, I have the same problem! How did you solve it?

@OriginalDREiD
Copy link
Author

@elclon unable to solve. Hoping @christiandelbianco can help me out.

@igitur
Copy link
Contributor

igitur commented Jun 18, 2018

This is a regression introduced by 51847dc .

@christiandelbianco It seems like you very deliberately moved the this.ConversationHandle = this.BeginConversation(sqlCommand); to after the preparation statements, but some of those preparation statements depend on the conversation GUID.

igitur referenced this issue Jun 18, 2018
@OriginalDREiD
Copy link
Author

Thank you @igitur, moving the block back to the original position resolved the issue for me.

@igitur
Copy link
Contributor

igitur commented Jun 18, 2018

I don't think this issue should be closed until @christiandelbianco comments on it.

@OriginalDREiD OriginalDREiD reopened this Jun 18, 2018
@christiandelbianco
Copy link
Collaborator

@igitur 👍

@christiandelbianco
Copy link
Collaborator

I immediately move this.ConversationHandle = this.BeginConversation(sqlCommand); where it was before

@igitur
Copy link
Contributor

igitur commented Jun 18, 2018

Great, thanks @christiandelbianco . PS: I discovered this project only last week. Great work!

@christiandelbianco
Copy link
Collaborator

@igitur As you found the resolution , i will put in Contributors page, if you do not mind

@igitur
Copy link
Contributor

igitur commented Jun 18, 2018

As you found the resolution , i will put in Contributors page, if you do not mind

No problem. I'm planning to submit a PR soon anyway.

christiandelbianco pushed a commit that referenced this issue Jun 18, 2018
@christiandelbianco
Copy link
Collaborator

Fix in version 7.5.0

@christiandelbianco
Copy link
Collaborator

@igitur : https://github.com/christiandelbianco/monitor-table-change-with-sqltabledependency/wiki/Contributors

@OriginalDREiD
Copy link
Author

Thank you, gentlemen.

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

No branches or pull requests

4 participants