Skip to content

Adding lag recipe #56

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Dec 21, 2021
Merged

Adding lag recipe #56

merged 3 commits into from
Dec 21, 2021

Conversation

ftisiot
Copy link
Contributor

@ftisiot ftisiot commented Dec 14, 2021

Adding lag recipe to demonstrate how to retrieve the previous value of an item. I believe it's useful for people willing to compare previous items on a stream

@ftisiot
Adding lag recipe
e0f927e 
Adding `lag` recipe to demonstrate how to retrieve the previous value of an item
@MartijnVisser MartijnVisser self-requested a review December 15, 2021 09:09
MartijnVisser
MartijnVisser reviewed Dec 21, 2021
View reviewed changes
Copy link
Contributor

@MartijnVisser MartijnVisser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much for creating the PR for this SQL recipe, it's much appreciated. I've left a couple of comments, let me know what you think and we should be able to merge it this week.

MartijnVisser
MartijnVisser approved these changes Dec 21, 2021
View reviewed changes
Copy link
Contributor

@MartijnVisser MartijnVisser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks greats, thanks for the recipe!

@MartijnVisser MartijnVisser merged commit b33313c into ververica:main Dec 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@MartijnVisser MartijnVisser MartijnVisser approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ftisiot @MartijnVisser