Civcraft / Citadel

Do not open issues here; open them on the maintained fork @ DevotedMC
https://github.com/DevotedMC/Citadel
BSD 3-Clause "New" or "Revised" License
6 stars 23 forks source link

Inefficient index / database usage #127

Closed psygate closed 8 years ago

psygate commented 8 years ago

Indexes are declared but never used in queries. See [CitadelReinforcementData)[https://github.com/Civcraft/Citadel/blob/master/src/vg/civcraft/mc/citadel/database/CitadelReinforcementData.java]. Either use indexes in queries explicitly, or merge the split up tables.

rourke750 commented 8 years ago

I don't believe you need to declare index use. From our timing it seems like indexs work properly.

suirad commented 8 years ago

@rourke750 should this remain open then?

ProgrammerDan commented 8 years ago

No.