Michael Manfre manfre

manfre commented on commit ovangle/django@bf3eef36a7
Michael Manfre

Why was 37 chosen as a shift value? This needs a code comment.

Michael Manfre

As an update. The patch I provided didn't scale well enough for my needs. I ended up passing messages to dedicated greenlets to forward to kafka.

Michael Manfre
  • Michael Manfre ed941ef
    Fixed signature of BaseDatabaseOperations.date_interval_sql()
manfre pushed to master at django/django
Michael Manfre
  • Michael Manfre 7fa7dd4
    Fixed signature of BaseDatabaseOperations.date_interval_sql() and doc…
manfre starred jquast/x84
manfre commented on issue aaugustin/django-pymssql#2
Michael Manfre

This may or may not be a pymssql specific error, but relying upon "schema].[table" is not recommended or officially supported by django-mssql.

manfre commented on pull request django/django#3899
Michael Manfre

My vote is for DatabaseWrapper should to remain in base.py.

Michael Manfre

Font choice may be part of the problem, but definitely cannot be the only change to make this readable for everyone.

Michael Manfre

Subtle color changes won't really help make it stand out or multiple word literals be cohesive. Giving them the same background color as .highlight

Michael Manfre
Inlined code/literals are difficult to distinguish
Michael Manfre

@jezdez that's a good compromise. IE8 should be supported just enough to make it so they can at least read the text on the site and click the impor…

manfre commented on pull request django/django#3609
Michael Manfre

I think @chicheng means doing the equivalent of {{cache.get_or_set('some-timestamp-key', datetime.datetime.now)}}