By default columns should not have indexes

[CaselIT]

Fixes #9

[uolot]

@tiangolo Are there any plans to merge this? I'd love to start using SQLModel in my projects, but this is a blocker.

[hidaris]

@tiangolo ping...

[perseghini]

@tiangolo any updates on this issue? Seems like this PR is fixing a typo which is creating indexes on all columns. It'd nice to have this merged so I don't have to manually edit all my Alembic migration files and remove those extra indexes.

[rabinadk1]

Fixes #152

[mkarbo]

@tiangolo update on merge?

[FFX01]

Any ETA on when this might be merged and released on PyPI? Seems like a pretty straight-forward and simple change. Are there concerns about it breaking something?

[heyaco]

Still no progress? :(

[tiangolo]

Thank you @CaselIT! 🚀

And thanks everyone for the discussion.

I didn't want to merge this without documenting it, adding tests, updating the examples and docs, etc.

I did all that in #205, the new docs explaining indexes are here: https://sqlmodel.tiangolo.com/tutorial/indexes/

Indexes are now opt-in and not created by default. You can read more about it all in that PR.

It is now released as part of SQLModel 0.0.6. 🎉

Given that, I'll close this PR now. ☕