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 support for Network address types for PostgreSQL #1078

Closed
wants to merge 2 commits into from

Conversation

Liewyec
Copy link

@Liewyec Liewyec commented Sep 14, 2023

Hello,
I noticed a missing support for Network address types for PostgreSQL. I added the support and they are returned s strings.

@vadz
Copy link
Member

vadz commented Oct 11, 2023

Sorry, I've somehow missed this. I see no problem with applying this, but I wonder if we should add support for the other related types too, see

https://github.com/postgres/postgres/blob/b6a77c6a6ccf698787201b001cbbbf9c89fe5715/src/include/catalog/pg_type.dat#L256-L265

?

@Liewyec
Copy link
Author

Liewyec commented Oct 18, 2023

Hello, sorry for the late answer, I was a bit busy and sick. I will look into this and add it, it looks easy and unless I am mistaken there are just 2 more types to add. I will update the pull request probably over weekend.

@vadz
Copy link
Member

vadz commented Dec 11, 2023

@Liewyec do you still plan to do it? TIA!

@Liewyec
Copy link
Author

Liewyec commented Dec 12, 2023

Hello, yes I am planing to do this, I should have a bit less work in next few weeks. Sorry it takes so long a I had a lot of other work.

@Liewyec
Copy link
Author

Liewyec commented Jan 2, 2024

Hello, I added the two more types, sorry it was taking so long.

@vadz
Copy link
Member

vadz commented Jan 2, 2024

Thanks, will merge soon.

@vadz vadz closed this in 65a7e4d Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants