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

stapi::refresh_ctable does not handle SQL generation errors #52

Closed
mutability opened this issue Mar 17, 2016 · 5 comments
Closed

stapi::refresh_ctable does not handle SQL generation errors #52

mutability opened this issue Mar 17, 2016 · 5 comments
Assignees

Comments

@mutability
Copy link

If gen_refresh_ctable_sql returns 0 to indicate an error, refresh_ctable does not handle this and tries to execute it as SQL, which fails:

03/17/2016 12:05:15 speedtable refresh of airline_fleets failed: ERROR:  syntax error at or near "0"
LINE 1: 0
        ^
@resuna resuna self-assigned this Mar 27, 2016
@resuna
Copy link
Member

resuna commented Apr 3, 2016

I've got a fix for this in the stapi-fixes branch.

What was the root cause of gen_refresh_ctable_sql returning zero? I'm not seeing how that could happen without a problem elsewhere.

@mutability
Copy link
Author

The cause was #53

@resuna
Copy link
Member

resuna commented Apr 3, 2016

Ah, OK. This should make that kind of failure easier to debug anyway.

I'll get started on #53 - that's not a use case I had ever considered.

@resuna
Copy link
Member

resuna commented May 27, 2016

I have rebased the stapi-fixes branch up to master. Can you please verify that it fixes this problems so I can merge it back?

@resuna
Copy link
Member

resuna commented Jul 11, 2016

Fixed in stapi-fixes.

@resuna resuna closed this as completed Jul 11, 2016
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