5964 fixes flyway for gbr split

[landreev]

New Contributors

Welcome! New contributors should at least glance at CONTRIBUTING.md, especially the section on pull requests where we encourage you to reach out to other developers before you start coding. Also, please note that we measure code coverage and prefer you write unit tests. Pull requests can still be reviewed without tests or completion of the checklist outlined below. Thanks!

Related Issues

• closes guestbookresponse table: "ERROR: column "downloadtype" does not exist" after pull request #5863 was merged #5964

Pull Request Checklist

• Unit tests completed
• Integration tests: None
• Deployment requirements, SQL updates, Solr updates, etc.: None
• Documentation completed
• Merged latest from "develop" branch and resolved conflicts

[coveralls]

Coverage remained the same at 19.676% when pulling c1f533a on 5964-fixes-flyway-for-gbr-split into f5cc5d0 on develop.

[pdurbin]

@landreev @scolapasta @sekmiller over at http://irclog.iq.harvard.edu/dataverse/2019-06-26#i_98757 @poikilotherm just asked if we've considered working on this issue:

Flyway: disable DDL generation from EclipseLink #5871

[pdurbin]

I just ran ec2-create-instance.sh -b 5964-fixes-flyway-for-gbr-split and 1fcb065 deployed just fine to EC2. Here's a screenshot:

I'm eager to get deployments to phoenix working again because right now we can't run the API test suite so I'm dragging this to QA.

Once again, there's a suggestion from @poikilotherm that we take a look at #5871 as a better solution. Or we could throw out all the Flyway stuff if it's driving us crazy. 😄

[pdurbin]

Deploys to EC2 as of 1fcb065. Approved. I left a long comment at #5966 (comment)