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

copy of pr #56 for testing #113

Closed
wants to merge 6 commits into from

Conversation

davecramer
Copy link
Collaborator

This is just for testing. If it passes here, I will push #56 rather than try to get it to build by adding a fake push

@davecramer
Copy link
Collaborator Author

Yes, I saw that. I'm looking at it now

@AndreMikulec
Copy link
Collaborator

@AndreMikulec
Copy link
Collaborator

@ikasou

May you take a look at? . . .

The opt_window_frame regression test is not passing (itself)
https://github.com/postgres-plr/plr/blob/e4e8887cd46cbc3237abd8108ccc62ab212b4c4f/expected/opt_window_frame.out

The expected result is with or without row 10.
https://ci.appveyor.com/project/davecramer/plr-lek5y/builds/39330178/job/yg5akbwatb896e9k?fullLog=true#L2597

@ikasou
Copy link

ikasou commented May 26, 2021

@ikasou

May you take a look at? . . .

The opt_window_frame regression test is not passing (itself)
https://github.com/postgres-plr/plr/blob/e4e8887cd46cbc3237abd8108ccc62ab212b4c4f/expected/opt_window_frame.out

The expected result is with or without row 10.
https://ci.appveyor.com/project/davecramer/plr-lek5y/builds/39330178/job/yg5akbwatb896e9k?fullLog=true#L2597

Looks like some test setup issue? (database pl_regression missing etc is the error message I see - am I not looking at the right place?)

@ikasou
Copy link

ikasou commented May 26, 2021

@ikasou
May you take a look at? . . .
The opt_window_frame regression test is not passing (itself)
https://github.com/postgres-plr/plr/blob/e4e8887cd46cbc3237abd8108ccc62ab212b4c4f/expected/opt_window_frame.out
The expected result is with or without row 10.
https://ci.appveyor.com/project/davecramer/plr-lek5y/builds/39330178/job/yg5akbwatb896e9k?fullLog=true#L2597

Looks like some test setup issue? (database pl_regression missing etc is the error message I see - am I not looking at the right place?)

Scrap that, it is clearly not relevant. I will take a closer look.

Makefile Show resolved Hide resolved
@davecramer davecramer closed this May 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants