Opened 12 years ago

Closed 11 years ago

#127 closed enhancement (fixed)

Change datatype of OId to long

Reported by: Dimitar Misev Owned by: Heinrich Stamerjohanns
Priority: major Milestone: Future
Component: rasserver Version: 8.3
Keywords: Cc: Peter Baumann, Dimitar Misev
Complexity: Very Hard

Description (last modified by Dimitar Misev)

OIds in rasdaman internally are represented as floating points, and it would be much better if they are actually integers. From a quick search, it seems like it might be sufficient to modify this

typedef double OIdPrimitive;

in reladminif/oidif.hh to long, and accordingly update the code that depends on it (which doesn't seem that much).

Change History (5)

comment:1 by Dimitar Misev, 12 years ago

It may be just a bug in the result printout in applications/rasql/rasql.cc but it's worth replacing the double OIDs with long in the code.

From a quick search I did, it seems like the the

typedef double OIdPrimitive;

in reladminif/oidif.hh should be converted to long.

comment:2 by Dimitar Misev, 12 years ago

Description: modified (diff)
Reporter: changed from beccati@… to Dimitar Misev
Summary: A double oid printed by rasql → Change datatype of OId to long

comment:3 by Dimitar Misev, 12 years ago

Type: defect → enhancement

comment:4 by Peter Baumann, 11 years ago

Milestone: → Future

comment:5 by Heinrich Stamerjohanns, 11 years ago

Complexity: → Very Hard
Resolution: → fixed
Status: new → closed

This issue has been addressed by the following fixes:
Expand r_Range to long long
OId-OIdCounter missing bracket fix
OId::OIdCounter is expanded to long long. Blobformats updated accordingly.

Note: See TracTickets for help on using tickets.