Gerando md5 para chaves de cache muito grandes

[fabiobatalha]

fix #19

[swarzesherz]

Sería mejor crear hash para todo y regresarlos directamente y tal vez utilizar sha1 en lugar de md5
return hashlib.sha1(finalkey).hexdigest()

[fabiobatalha]

Yo pense en generar el hash solamente para los que exceden los bytes puesto que generar hash es un poco costoso, pero pondemos generar el hash para todos. Que piensas?

[swarzesherz]

Pues creo que por uniformidad estaría mejor, que sólo aplicar a las cadenas
que superen el límite
El El vie, 13 de noviembre de 2015 a la(s) 14:00, Fabio Batalha <
notifications@github.com> escribió:

In analytics/utils.py
#20 (comment):

@@ -26,7 +27,12 @@ def generate_key(_the_args, *_the_kwargs):
key += [str(i) for i in the_args[1:]]
key.append(str(the_kwargs))

•    return "_".join(key)
  

•    finalkey = "_".join(key)
  

•    if len(finalkey) > 249:
  

•        return hashlib.md5(finalkey).hexdigest()
  

Yo pense en generar el hash solamente para los que exceden los bytes
puesto que generar hash es un poco costoso, pero pondemos generar el hash
para todos. Que piensas?

—
Reply to this email directly or view it on GitHub
https://github.com/scieloorg/analytics/pull/20/files#r44827364.

[fabiobatalha]

Ok, voy a cambiar.

http://www.sami-lehtinen.net/blog/python-hash-function-performance-comparison

Que piensas de utilizar el hash()

[swarzesherz]

Creo que para este caso será mejor utilizar un algoritmo ya sea md5 o sha
como que hash() no me convence mucho
El El vie, 13 de noviembre de 2015 a la(s) 14:11, Fabio Batalha <
notifications@github.com> escribió:

In analytics/utils.py
#20 (comment):

@@ -26,7 +27,12 @@ def generate_key(_the_args, *_the_kwargs):
key += [str(i) for i in the_args[1:]]
key.append(str(the_kwargs))

•    return "_".join(key)
  

•    finalkey = "_".join(key)
  

•    if len(finalkey) > 249:
  

•        return hashlib.md5(finalkey).hexdigest()
  

Ok, voy a cambiar.

http://www.sami-lehtinen.net/blog/python-hash-function-performance-comparison

Que piensas de utilizar el hash()

—
Reply to this email directly or view it on GitHub
https://github.com/scieloorg/analytics/pull/20/files#r44828511.

[fabiobatalha]

Hash es deterministico tanto cuanto md5 y sha1.

[swarzesherz]

Perdón por la tardanza estaba en otros asuntos, respecto a la velocidad me parece bien y viendo mejor dentro de python el manejo se ajusta al objeto como tal, me parece bien (Y)