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

Forward-slash used as division operator within PL/SQL causes migration to fail #2493

Closed
alextercete opened this issue Sep 2, 2019 · 1 comment

Comments

@alextercete
Copy link
Contributor

@alextercete alextercete commented Sep 2, 2019

Which version and edition of Flyway are you using?

Flyway 6.0.1

Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)

CLI

Which database are you using (type & version)?

Oracle 19.3

Which operating system are you using?

Windows

What did you do?

Run migration:

CREATE OR REPLACE FUNCTION divide(a NUMBER, b NUMBER) RETURN NUMBER IS

BEGIN

    RETURN a/b;

END;

/
What did you expect to see?

Migration succeeds.

What did you see instead?

Migration fails. However, if a/b is surrounded by parenthesis, as in (a/b), the migration succeeds.

@alextercete alextercete added this to the Flyway 6.0.2 milestone Sep 2, 2019
@alextercete alextercete changed the title Forward-slash used as division operator with PL/SQL causes migration to fail Forward-slash used as division operator within PL/SQL causes migration to fail Sep 2, 2019
@alextercete alextercete reopened this Oct 21, 2019
@alextercete
Copy link
Contributor Author

@alextercete alextercete commented Oct 21, 2019

Looks like Flyway 6.0.7 reintroduces this issue when there are spaces around the /:

CREATE OR REPLACE FUNCTION divide(a NUMBER, b NUMBER) RETURN NUMBER IS

BEGIN

    RETURN a / b;

END;

/

Loading

@alextercete alextercete removed this from the Flyway 6.0.2 milestone Oct 21, 2019
@alextercete alextercete added this to the Flyway 6.1.0 milestone Oct 21, 2019
@alextercete alextercete removed this from the Flyway 6.1.0 milestone Oct 21, 2019
@alextercete alextercete added this to the Flyway 6.0.8 milestone Oct 21, 2019
dohrayme pushed a commit to dohrayme/flyway that referenced this issue Feb 3, 2020
dohrayme pushed a commit to dohrayme/flyway that referenced this issue Feb 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant