TAPRegExt function declaration [was: ADQL polymorphic functions]

Grégory Mantelet gregory.mantelet at astro.unistra.fr
Mon Apr 26 13:06:28 CEST 2021


Dear Markus and DAL,

It is indeed a nice simplification that would work for these use-cases. 
I actually did not think to this as a solution when I mentioned it :D

I just hope it won't break anything though...but as said, it is just 
informative for clients/users, so I don't see how it could break 
something. I would have to adapt a bit VOLLT/ADQL-Lib to make it works 
completely with this simplified "type system" (i.e. 'string' and 
'numeric' would work in the current state but not 'geometry' and 'any') 
so that the query check process of TOPCAT works as expected. But that 
would apparently be a trivial modif. on my side.

So, it's ok for me. And if ok for others as well, I am totally fine with 
you, Markus, to write this PullRequest.

Cheers,
Grégory


On 23/04/2021 18:10, Markus Demleitner wrote:
> Dear Grégory,
>
> [restricting to DAL, as this is becoming rather DAL-specific]
>
> On Fri, Apr 23, 2021 at 10:37:03AM +0200, Grégory Mantelet wrote:
>>       not the best one.
>>       An alternative could be to introduce special types like:
>> ANY, ANY_NUMERIC, ...
>>
>> About datatypes in UDF signature, for the moment, nothing specifies what
>> datatypes are allowed. And this is especially the case in ADQL, where there
>> is
>> no strong typing.
>>
>> Personally, when reading such datatypes, my ADQL parser simplifies them as
>> "numeric", "string", "geometry" and "unknown" (a synonym of "any") in order
>> to
>> check types ; it allows it to make a simple query check with approximate
> You know -- shouldn't we just adopt this as the recommended type
> system (with "any" instead of "unknown")?  I *think* that's covering
> the use cases we have for the signatures, and it's nice and simple.
>
> I'd totally support an issue to this effect against
> https://github.com/ivoa-std/TAPRegExt.  I'd even write a PR unless
> you want to.
>
> Thanks,
>
>          Markus



More information about the dal mailing list