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

Added support for HP Vertica (tested on HP Vertica 6.5 and 7). #740

Merged
merged 5 commits into from May 20, 2014

Conversation

@nathanvick
Copy link
Contributor

@nathanvick nathanvick commented Apr 29, 2014

I added support for HP Vertica, issue #344, and ran some basic tests. HP Vertica uses a similar syntax and built-in functions to PostgreSQL, but it deviates in some areas (v_catalog instead of information_schema, BEGIN/END instead of $$) and has a few unsupported features (no indexes).

I'm not aware of a published list of differences between PostgreSQL and Vertica. However, I still based the tests on the PostgreSQL database support, with modifications where required.

@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Apr 29, 2014

Hi Nathan,

Thank you very much for this! Would you be willing to support this in case issues come up or changes have to be made in the future (Flyway has been pretty stable, but you never know)?

Could you also contribute a pull request for flywaydb.org (flyway-3.1 branch) with the docs? This would be a database page, as well as an update to the contribute section with details on how to setup the dev environment for Vertica.

Cheers
Axel

@nathanvick
Copy link
Contributor Author

@nathanvick nathanvick commented Apr 29, 2014

Sure, I'll work on the Vertica docs in flywaydb.org. I'd be happy to contribute patches, if you hear of any issues.

@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Apr 29, 2014

Sounds good! Looking forward to it!

@axelfontaine axelfontaine added this to the Flyway 3.1 milestone Apr 29, 2014
Nathan Vick added 4 commits Apr 29, 2014
axelfontaine added a commit that referenced this pull request May 20, 2014
Added support for HP Vertica (tested on HP Vertica 6.5 and 7).
@axelfontaine axelfontaine merged commit ddd2f9c into flyway:master May 20, 2014
1 check passed
1 check passed
continuous-integration/travis-ci The Travis CI build passed
Details
@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented May 20, 2014

Thank you so much Nathan! Finally merged it. I tried to get Vertica to work on Windows (my dev machine), but the install was broken out of the box. Go figure...

So I moved the Vertica support to a new category: ContributorSupportedDB . The tests can be run using mvn -PContributorSupportedDBTest

Cheers
Axel

P.S.: I've added to the hall of fame page!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.