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

Pour gagner quelques msc #5

Open
manland opened this issue Nov 14, 2016 · 6 comments
Open

Pour gagner quelques msc #5

manland opened this issue Nov 14, 2016 · 6 comments

Comments

@manland
Copy link

manland commented Nov 14, 2016

Si jamais @cheleb te rattrape :

  • à la fin du fetch prépare quelques random :
cache.radom[0] = [];
cache.random[1] = getRandomIds(attendees.length, 1);
cache.random[2] = getRandomIds(attendees.length, 2);
cache.random[3] = getRandomIds(attendees.length, 3);
cache.random[4] = getRandomIds(attendees.length, 4);

Par la même pré-jsonify :

cache....then(winners => winners.map(({profile: {first_name: first_name, last_name: last_name}}) => { return {first_name: first_name, last_name: last_name}}))

Voir même pré-stringify :

cache.....then(JSON.stringify(winners))

et donc à la fin tu peu directement :

app.get('/winners', function(req, res){
  const nb_winner = req.query.nb
  if (nb_winner == undefined || isNaN(nb_winner) || nb_winner < 0) {
    res.status(400)
    res.json({error: 'Nb param should be a positive integer'})
  } else if (nb_winner == 0) {
    res.json([])
} else {
res.header('application/json')
res.send(cache[nb_winner])
}

Je pense que tu dois pouvoir viser les 30msc ;)

@fteychene
Copy link
Contributor

Yes c'est ce qu'on s'est dis avec olivier cet aprem xD

C'est vraiment pour essayer de se la mettre.

IL m'as mm dit qu'il allais peut-tere préparer un DDos sur le tirage au sort pour etre sur que je soit plus lent :D

@manland
Copy link
Author

manland commented Nov 14, 2016

Hahaha on en parlait au dernier jug aussi de ddos les autre implem :D

@mackristof
Copy link
Member

attention a ce jeu là je vais sortir les channels pour pré-fetch les winners :-p

@cheleb
Copy link
Member

cheleb commented Nov 14, 2016

Je viens de le faire... Rendu bofbif je pense que ma virtualisation est le
truc le plus lourd en fait... Ou le réseaux...
À++

On Mon, 14 Nov 2016, 23:03 mackristof, [email protected] wrote:

attention a ce jeu là je vais sortir les channels pour pré-fetch les
winners :-p


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#5 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXGq1E9szIF0hOmtMs3da1abyHlSAkSks5q-NqZgaJpZM4Kxmtr
.

@mackristof
Copy link
Member

je confirme le pré-fetch des winners change rien. je pense que l'on arrive au bout de l'histoire.

@cheleb
Copy link
Member

cheleb commented Nov 15, 2016

+1

On Tue, 15 Nov 2016, 21:49 mackristof, [email protected] wrote:

je confirme le pré-fetch des winners change rien. je pense que l'on arrive
au bout de l'histoire.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#5 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXGq3l7ERX1RbmsbtQfQa5OwXZdmQbZks5q-hrCgaJpZM4Kxmtr
.

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

No branches or pull requests

4 participants