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

Assertion fails on line 87 of table_connector.py #5

Open
gabegaster opened this issue May 19, 2016 · 5 comments
Open

Assertion fails on line 87 of table_connector.py #5

gabegaster opened this issue May 19, 2016 · 5 comments

Comments

@gabegaster
Copy link
Contributor

message_id, chat_id, and phone_or_mail don't form a composite key. Assertion error fails here.

@mdezube
Copy link
Owner

mdezube commented May 19, 2016

I'm not sure what you mean, did this actually fail for you? Or are you saying it needs to be cleaned such as what you did in your PR?

@gabegaster
Copy link
Contributor Author

It actually failed for me.

gabe

On May 19, 2016, at 7:19 PM, mdezube [email protected] wrote:

I'm not sure what you mean, did this actually fail for you? Or are you saying it needs to be cleaned such as what you did in your PR?


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

@mdezube
Copy link
Owner

mdezube commented May 22, 2016

Interesting, can you see what/how many dupes you have? Perhaps it's due to failed messages being resent, or messages sending over sms and imessage. Either way, we should drop the records and print out how many had this issue.

@mdezube
Copy link
Owner

mdezube commented Jun 12, 2016

So I'd still like to understand why you have duplicates. Let's dive in and debug one day.

@mdezube
Copy link
Owner

mdezube commented Jan 20, 2017

Can you still replicate this? I fixed an issue with group messages that I think resolves the cause.

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

2 participants