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

Add UnboxDatum for Range<T> #1827

Merged
merged 2 commits into from
Aug 24, 2024

Conversation

mhov
Copy link
Contributor

@mhov mhov commented Aug 24, 2024

I was getting compiler complaints from this input type my_func(arr: Array<Range<Date>>) and calling .get_by_name::<Range<Date>>. Implement the required trait.

@workingjubilee workingjubilee changed the title Adding UnboxDatum for Range<T> Add UnboxDatum for Range<T> Aug 24, 2024
@workingjubilee
Copy link
Member

Thanks!

@workingjubilee workingjubilee merged commit ed18e63 into pgcentralfoundation:develop Aug 24, 2024
14 checks passed
eeeebbbbrrrr added a commit that referenced this pull request Sep 6, 2024
Welcome to pgrx v0.12.2. This is a minor release that fixes a few bugs,
improves compilation times with `cargo pgrx run/install/test`, and adds
a few more Postgres headers.

As usual, please `cargo install cargo-pgrx --version 0.12.2 --locked`.
Then you can run `cargo pgrx upgrade` in your extension crate's root to
update its dependencies.

## What's Changed

* [ `CPPFLAGS`] Switch to `USE_ASSERT_CHECKING` by @SamuelMarks in
#1826
* Add UnboxDatum for Range<T> by @mhov in
#1827
* Moved Sized from FromDatum methods to trait by @YohDeadfall in
#1831
* Used SPI result type for cursor API by @YohDeadfall in
#1836
* always compile `pgrx_embed_*` without opts by @eeeebbbbrrrr in
#1838
* Fixed out of bounds reads for open_cursor by @YohDeadfall in
#1837
* Include `access/visibilitymap.h` and `utils/tuplestore.h` by
@eeeebbbbrrrr in #1841

## New Contributors

Thanks to these folks for their first-time contributions -- it's greatly
appreciated!

* @SamuelMarks made their first contribution in
#1826
* @YohDeadfall made their first contribution in
#1831

**Full Changelog**:
v0.12.1...v0.12.2
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.

2 participants