Skip to content

Conversation

@SteveLauC
Copy link
Contributor

Which issue does this PR close?

Rationale for this change

As requested by @l1t1 in this comment.

What changes are included in this PR?

Document generate_series() as an alias of range().

Are these changes tested?

Are there any user-facing changes?

no

Copy link
Member

@jonahgao jonahgao left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Copy link
Contributor

@Jefffrey Jefffrey left a comment

Choose a reason for hiding this comment

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

Thanks for this

Would be nice if it were in alphabetical order with the other items 👍

@SteveLauC SteveLauC force-pushed the docs/range_alias_generate_series branch from efe742c to 3bb4836 Compare February 23, 2024 09:49
@SteveLauC
Copy link
Contributor Author

Would be nice if it were in alphabetical order with the other items 👍

Indeed! done.

@Jefffrey Jefffrey merged commit 3b355c7 into apache:main Feb 23, 2024
@Jefffrey
Copy link
Contributor

Cheers 🚀

@SteveLauC SteveLauC deleted the docs/range_alias_generate_series branch February 23, 2024 09:59
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.

3 participants