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

SQL : Nouvel identifiant entier pour les catégories et les flux #262

Closed
Alkarex opened this issue Nov 11, 2013 · 5 comments
Closed

SQL : Nouvel identifiant entier pour les catégories et les flux #262

Alkarex opened this issue Nov 11, 2013 · 5 comments
Assignees
Milestone

Comments

@Alkarex
Copy link
Member

Alkarex commented Nov 11, 2013

Je propose d'utiliser un simple SMALLINT(voir #258) pour les identifiants de catégories et de flux.

Ces identifiants sont utilisés dans les jointures les plus coûteuses.

De plus, je suggère de choisir cet identifiant comme étant max(id)+1, car le hachage me semble apporter uniquement des problèmes et pas d'avantage dans ce cas.

Par ailleurs, un UNIQUE() sur category.name et feed.url me semble nécessaire.

@Alkarex
Copy link
Member Author

Alkarex commented Nov 11, 2013

Dans cette perspective, j'ai passé feed.url en VARCHAR - nécessaire pour UNIQUE -, sans compter le fait que c'est un champ toujours retourné par les requêtes principales 5af7c47

@ghost ghost assigned Alkarex Nov 18, 2013
Alkarex added a commit that referenced this issue Nov 18, 2013
Implémentation de
#262
La catégorie par défaut à le numéro 1.
Les numéros de catégories et de flux sont automatiques (1, 2, 3...)
L'installeur semble marcher.
@Alkarex
Copy link
Member Author

Alkarex commented Nov 18, 2013

La catégorie par défaut ("Sans Catégorie") a le numéro 1.
Les numéros de catégories et de flux sont automatiques (1, 2, 3...).
L'installeur semble marcher.

@Alkarex Alkarex closed this as completed Nov 18, 2013
Alkarex added a commit that referenced this issue Nov 19, 2013
@Alkarex
Copy link
Member Author

Alkarex commented Nov 19, 2013

Script de mise à jour adapté en conséquence #255

@Alkarex
Copy link
Member Author

Alkarex commented Nov 19, 2013

Install.php semble bien fonctionner suite à ces changements #273

@Alkarex
Copy link
Member Author

Alkarex commented Nov 19, 2013

Boh, je suis impressionné par le gain de vitesse :-) Testé avec 60k articles sur un serveur Atom.

Alkarex added a commit that referenced this issue Nov 19, 2013
Suite au nouvel identifiant entier pour les catégories et les flux
#262
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

1 participant