Re: AbstractJdbc2Array - another patch

Kris Jurka pisze:
> Attached is a revised version of your patch after I worked on it a bit. 
> Things I've changed:
> 
> 1) Looking up the array type for a base type by using typelem doesn't 
> work because typelem is not unique.  Since no code was calling 
> getPGTypeArray, I just removed this code.
Agree.

> 
> 3) Fixed the regression tests and added new ones to verify that array 
> handling works.
I wanted to do it, but I was waiting for you comments about my code.
Thanks for that.

> 4) Changed Array.getResultSet to use generic coding rather than 
> hardcoding type oids for each java.sql.Types value.  This allows 
> getResultSet to be used on types that aren't known to the driver, like 
> aclitem[].  You still can't call getArray on this type, but we're 
> getting closer.
Agree.

> 
> 5) Array.getResultSet didn't respect index or offset parameters.
Sorry, my mistake.

> 
> 6) For TypeInfoCache I've added a new column to link the base and array 
> types instead of duplicating the whole rows.
Agree.

> 
> 7) Code did not correctly handle a string value of NULL in arrays for 
> 8.1 and earlier server versions.
I was concentrate on make it working for >= 8.2 - again my mistake, sorry.

> So with these changes, I think the code is pretty much ready to go, with 
> the exception of how multidimensional results are returned.  The way you 
> wrap the arrays in Object[] makes it impossible to cast to things like 
> Integer[][].  Also I think we should be able to return multidimensional 
> arrays of primitive type if the "compatible" option is set to 8.2.  I 
> think you should be using java.lang.reflect.Array#newInstance to create 
> the arrays you are returning to get the correct type instead of building 
> them up incrementally.
Well, I aware that java.lang.reflect.Array#newInstance can be used
however I didn't know for what versions of JDK reflect package can be 
used - if you have mentioned about it so I guess that I can - in the 
attachment you can find AbstractJdbc2Array patch - please review it.

I one to discuss one more thing - see below lines in the AbstractJdb2Array:
this.useObjects = connection.haveMinimumCompatibleVersion("8.3");
this.haveMinServer82 = connection.haveMinimumServerVersion("8.2");

We must clarify, when to use objects instead of primitive types ? Can 
you describe it ?

Regards,
Marek