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

BUG: Don't use SET LOCAL to allow redshift to work #1163

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@cpcloud
Member

cpcloud commented Oct 10, 2017

Closes #1158

@cpcloud cpcloud self-assigned this Oct 10, 2017

@cpcloud cpcloud added this to the 0.11.3 milestone Oct 10, 2017

@cpcloud

This comment has been minimized.

Member

cpcloud commented Oct 10, 2017

@FedericoV Can you give this a try and let me know if it fixes #1158?

@cpcloud cpcloud force-pushed the cpcloud:fix-tz-redshift branch from b79e0bc to d6fdd8c Oct 10, 2017

@FedericoV

This comment has been minimized.

FedericoV commented Oct 11, 2017

Hi Philip,

Thanks for all the work! I tried executing a few different queries and all worked without any exceptions. Any specific commands you want me to check?

@cpcloud

This comment has been minimized.

Member

cpcloud commented Oct 12, 2017

@FedericoV Nothing specific. Builds are passing, so I'm going to merge.

@cpcloud cpcloud closed this in 5b16ba5 Oct 12, 2017

@cpcloud cpcloud deleted the cpcloud:fix-tz-redshift branch Oct 12, 2017

@cpcloud

This comment has been minimized.

Member

cpcloud commented Oct 12, 2017

@FedericoV Thanks for reporting and testing!

@FedericoV

This comment has been minimized.

FedericoV commented Oct 12, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment