-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Add auto dist style #1885
Add auto dist style #1885
Conversation
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA. In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, don't hesitate to ping @drewbanin. CLA has not been signed by users: @tjengel |
The cla-bot has been summoned, and re-checked this pull request! |
@drewbanin What branch should I be rebasing to? |
You’ll want dev/louisa-may-alcott- make to pull from the
fishtown-analytics/dbt first as we made some more changes to this branch
today!
On Mon, Nov 4, 2019 at 8:28 PM Tyler Engel ***@***.***> wrote:
What branch should I be rebasing to?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1885>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AALYIE3Y2ZYCCA6DKWT5DB3QSDD5LANCNFSM4JIM5QDA>
.
--
Drew Banin
Fishtown Analytics
|
6f9df4b
to
e1b040c
Compare
@drewbanin This should be good to go now |
@tjengel this LGTM but something weird happened with the test suite. I just kicked these tests off again - will merge it when they pass :) |
Hi @tjengel, could you merge the latest dev/louisa-may-alcott in to this branch? It has a dependency fix that I think will get your tests happy again! |
…d_auto_dist_style
@beckjake This is done |
Merging this now, thanks for your contribution @tjengel! |
This pr adds 'auto' as a diststyle for redshift. Fixes issue #1882