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

pointing to specificname for procedures with same signature and support for cleaning temporary table too #1750

Merged
merged 5 commits into from Aug 24, 2017

Conversation

@franciskittu
Copy link

@franciskittu franciskittu commented Aug 21, 2017

#1167

P.S.: A little problem with build... I needed to comment the scanForClassesSubPackage() of the ClassPathScannerSmallTest class to correctly create the build package. (the issue raised right after forking, so with no personal modification, I decided to do a pull request anyway)

franciskittu added 2 commits Aug 21, 2017
Clean command matters...
-Manage "drop procedure" by specificname.
-Added support for dropping temporary tables.
@CLAassistant
Copy link

@CLAassistant CLAassistant commented Aug 21, 2017

CLA assistant check
All committers have signed the CLA.

@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Aug 21, 2017

@franciskittu ClassPathScannerSmallTest isssue should be resolved. Could you also add a testcase for this?

@franciskittu
Copy link
Author

@franciskittu franciskittu commented Aug 22, 2017

@axelfontaine Testcase added as required. Please check it. Thanks.

@axelfontaine axelfontaine merged commit 8cf34d0 into flyway:master Aug 24, 2017
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
license/cla Contributor License Agreement is signed.
Details
@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Aug 24, 2017

Merged. Thanks!

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

3 participants