Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#244 closed defect (worksforme)

Dashes should be allowed in collection names

Reported by: Dimitar Misev Owned by: Dimitar Misev
Priority: major Milestone: 8.4
Component: qlparser Version: 8.3
Keywords: Cc: Peter Baumann
Complexity: Medium

Description

Right now it seems that rasdaman doesn't allow (parsing fails I believe) any '-' in collection names.

To reproduce just try to create a collection name that contains '-', e.g.

create collection test-2 GreySet

Change History (6)

comment:1 by Peter Baumann, 11 years ago

indeed, collection names must be regular identifiers, as known from programming languages - see flex source in qlparser/. I haven't seen a requirement to change this, nor a comprehensive list of what should be allowed in addition - before changing anything here it should be great to see that.

The original idea during rasdaman design anyway was that metadata offer any kind of naming (incl. whitespace, national charsets, etc.), and collection names are just an internal interface. So maybe we can fix this by having service operators use a metadata table (which might be good for many purposes) defined by them?

comment:2 by Peter Baumann, 11 years ago

Milestone: 9.0

comment:3 by Dimitar Misev, 11 years ago

It seems that tables in SQL similarly can't have '-' in the name. The way they go to allow such extra characters is typically by quoting the name, and always referring to it with quotes (e.g. in postgres).

We could go similarly about this, add another rule to the lexer to allow anything inbetween double quotes for a collection name?

The '-' question came up by EOX and MEEO if I remember correctly.

comment:4 by Dimitar Misev, 11 years ago

I'd guess that allowing '-', '.', etc. might confuse the lexer and result in some unwanted subtractions or similar issues.

comment:5 by Dimitar Misev, 11 years ago

Resolution: worksforme
Status: newclosed

Ok, I resolve this as "will be fixed in petascope by #310", whereas in rasql we stick to the current situation.

comment:6 by abeccati, 11 years ago

Complexity: Medium
Milestone: 9.08.4
Note: See TracTickets for help on using tickets.