eclipse-vertx / vertx-sql-client

High performance reactive SQL Client written in Java
Apache License 2.0
877 stars 195 forks source link

CIDR Support in vertx-pg-client #1418

Closed AyanKoche closed 2 months ago

AyanKoche commented 4 months ago

Motivation: CIDR Data type support in vertx-pg-client

Explain here the context, and why you're making that change, what is the problem you're trying to solve. There are several data types supported by vertx-pg-client but CIDR was missing implementation. As the project I am working on requires that support, contributing it is the better way. Changes:

Conformance:

You should have signed the Eclipse Contributor Agreement as explained in https://github.com/eclipse/vert.x/blob/master/CONTRIBUTING.md Please also make sure you adhere to the code style guidelines: https://github.com/vert-x3/wiki/wiki/Vert.x-code-style-guidelines

AyanKoche commented 4 months ago

Addressed the previous comments. Please re-review it. @vietj

vietj commented 4 months ago

can you squash your commits in a single one ?

AyanKoche commented 4 months ago

Sure ,doing it

AyanKoche commented 4 months ago

@vietj Hi sorry for delayed response, I have made the changes and squashed the commits into single commit. Kindly have a look at it

AyanKoche commented 3 months ago

Hi @vietj , any review comments on the PR?

AyanKoche commented 2 months ago

@vietj I have added test cases dependent on a real database. I have tested them with pg embedded version 12 as previous versions didn't have CIDR support. Please review and provide input.

vietj commented 2 months ago

great feature thanks @AyanKoche