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

Considering changing all the fk_columns from integer to smallint but also take into account reordering the columns #327

Open
tudorbarascu opened this issue Oct 1, 2020 · 3 comments

Comments

@tudorbarascu
Copy link
Member

Basically, we shouldn't have any fk_column as integer as smallint is enough, thus keeping two small columns in the same 4k page (default of postgres) and improving performance, index size etc.

https://medium.com/braintree-product-technology/postgresql-at-scale-saving-space-basically-for-free-d94483d9ed9a

@haubourg
Copy link
Contributor

haubourg commented Oct 2, 2020

@tudorbarascu yup, would be nice. Do you reach large enough database to feel such optimizations are necessary in your cases?

@ponceta
Copy link
Member

ponceta commented Nov 5, 2020

@tudorbarascu do you encounter size or latency issues with QWAT?

The technical idea seems good to me, I would assume that only materials would be possible to reach over the 32767 limit smallint (and now still far from it).

@tudorbarascu
Copy link
Member Author

@ponceta @haubourg The project is working well for now as we all fixed most of whatever issues I had over the years. However, I always considered that we should do the basics really well as the little performance hits propagate on the long run. For now, the biggest pain point is the attribute table performance for lots of features but this concerns the whole QGIS platform, not just QWAT/QGEP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants