casting fails with user-defined types

[Dimitar Misev]

Casting to a user-defined composite type fails, e.g. assuming a base type

struct CustomPixel { octet band1, band2, band3, band4; };

has been inserted with rasdl, the following statement fails:

insert into coll values (CustomPixel) inv_tiff($1)

[Peter Baumann]

Dimitar, do you think his is a cast or a "TIFF with 4 bands" problem?

[Dimitar Misev]

The casting itself is the issue, has nothing to do with the convertor function.

[Veranika Liaukevich]

Fixed in changeset:4383ed3570dbfb27ae9f5f983d72e723ed5aa831

[Dimitar Misev]

I submitted a patch with a test in source:systemtest/testcases_mandatory/test_conversion

(TestPixel) decode($1) seems to work fine, but (TestPixel) inv_tiff($1) fails with

Error: QtConversion::evaluate() no structure type compatible found

[Veranika Liaukevich]

Did you recompile code? Both selection and insertion with inv_tiff work for me, the test from your patch is passed.

[Dimitar Misev]

Hmm yes I did recompile, this is the output of my test:

test.sh: ----- user-defined type conversion ------
test.sh: creating collection... ok.
rasdaman error 2002: Internal error: Entry in user defined type not found.
test.sh: user-defined base type cast inf_tiff test failed.
test.sh: user-defined base type cast decode test passed.

[Veranika Liaukevich]

(By the way, you have a typo: "inV_tiff test passed").
That's strange:

test.sh: ----- user-defined type conversion ------
test.sh: creating collection... ok.
test.sh: user-defined base type cast inf_tiff test.
test.sh: user-defined base type cast decode test passed.

Does "select (TestPixel) inv_tiff($1)" also fail?

[Dimitar Misev]

Strange indeed, and select fails as well

rasql: rasdaman query tool v1.0, rasdaman v9.0.1-ge5210e4 -- generated on 13.05.2014 14:27:43.
opening database RASBASE at localhost:7001...ok
fetching type information for GreyString from database, using readonly transaction...ok
reading file /home/dimitar/projects/rasdaman/community/src/systemtest/testcases_mandatory/test_conversion/testdata/multiband.tif...ok
  constant 1: GMarray
  Oid...................: 
  Type Structure........: 
  Type Schema...........: marray< char >
  Domain................: [0:691]
  Base Type Schema......: char
  Base Type Length......: 1
  Data format.......... : Array
  Data size (bytes).... : 692
Executing retrieval query...rasdaman error aborting transaction...2002: Internal error: Entry in user defined type not found.
ok
rasql done.

[Dimitar Misev]

These are the last commits from git log

* e5210e4 (HEAD, master) ticket:119 - add conversion test
* 053b4fe (origin/master, origin/HEAD) ticket:707 - Hide debug output in StorageLayout
* 4383ed3 ticket:119 - Casting with user-defined types
* 251db02 ticket:210 - Fix null pointer dereference
...

[Veranika Liaukevich]

I have the same commits order, and the working directory is clean.

Try just "select inv_tiff($1)", the error message seems to appear during conversion, not casting.

[Dimitar Misev]

Yes indeed, still same error

Request: 'select inv_tiff(#MDD0#)'...parsing...checking semantics...evaluating...Error: QtConversion::evaluate() no structure type compatible found

[Veranika Liaukevich]

I took a look at QtConversion code, it doesn't create a new temprorary type after the conversion is done, but searches for an appropriate among already existing. On my system it succeeds, on yours fails, so we need to change this behaviour. I will have a look later.

[Dimitar Misev]

It seems that the inv_tiff returns a struct of chars, rather than of octets.
The TestPixel is a struct of 4 octets, which doesn't match a struct of 4 chars, so that's why it fails.
The input testdata file multiband.tif has char bands, after creating a multiband.tif with octet bands it works fine. Still puzzled how it worked for you though
In any case, it was an issue of the test and not the implementation, so reclosing the ticket.

[Veranika Liaukevich]

Wait, this is what casting is about: converting one types into others. You can convert char into octet without any problem (try "select (octet) 128"), the result should be 127 because of overflow. This is how it works now for base types, so I think it should work in the same way for attributes of structures.

decode($1) also returns structure of chars, which is later converted to structure of octets (TestPixel), and this works for you as well.

[Dimitar Misev]

Yes, so inv_tiff should actually create a temporary struct, if it cannot find the corresponding type in the database.

[Veranika Liaukevich]

Yes, exactly