Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OMetadata SYSTEM_CLUSTER collection is incomplete? #7182

Closed
rnagy90 opened this issue Feb 20, 2017 · 2 comments
Closed

OMetadata SYSTEM_CLUSTER collection is incomplete? #7182

rnagy90 opened this issue Feb 20, 2017 · 2 comments
Milestone

Comments

@rnagy90
Copy link

rnagy90 commented Feb 20, 2017

OrientDB Version: 2.2.15

Java Version: 1.8

OS: Win 8 + Linux

Maybe it's a bug maybe it's not, but the OMetadata#Set SYSTEM_CLUSTER contains the system class names as lower case.
There shouldn't be more class names? I found that when i create a new database, it automatically creates the OSequence, V, E and _studio classes.

I have one more question. I've just found out the JDBC driver call check this Set to get the table types with OClass#getName() without '.toLowerCase()'.
/* I'm going to fix it and update the test cases in my fork, because i need it now for my project and you can merge it if you want :). */
I am not sure, but the OPaginatedCluster constructor already checks the class name with the SYSTEM_CLUSTER.contains(name) and maybe it also has an incorrect result potential. (I didn't check the call hierarchy, it's just a guess)

@tglman
Copy link
Member

tglman commented Mar 28, 2017

hi @rnagy90,

I sorted out that, closing.

Regards

@tglman tglman closed this as completed Mar 28, 2017
@tglman tglman added this to the 2.2.x (next hotfix) milestone Mar 28, 2017
@rnagy90
Copy link
Author

rnagy90 commented Apr 2, 2017

hi @tglman!

Thank you for the fix, i'm going to test it.

@santo-it santo-it modified the milestones: 2.2.18, 2.2.x (next hotfix) Apr 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants