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.

Sign up for GitHub

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

Jump to bottom

Добавить поддержку long() в пропатченный json-backend #78

Open
byaka opened this issue Apr 22, 2016 · 3 comments
Open

Добавить поддержку long() в пропатченный json-backend #78

byaka opened this issue Apr 22, 2016 · 3 comments
Assignees
@byaka
Labels
high-priority testing
Milestone
Last major release

Comments

@byaka
Copy link
Owner

byaka commented Apr 22, 2016

Патчинг при помощи experimentalPackage подменяет json-backend на ujson.
К сожалению ujson не поддерживает long() и поддержка не предвидется (подробнее ultrajson/ultrajson#99).

Однако можно добавить поддержку через experimental.asyncJSON_dumps(), у него нет проблем с поддержкой типов. Если сделать отлов ошибок у ujson, можно сделать автоматическое переключение при неудаче.
@byaka byaka self-assigned this Apr 22, 2016
@byaka byaka added this to the Last major release milestone May 21, 2016
@byaka
Copy link
Owner Author

byaka commented May 22, 2016 •
edited

Обнаружен неприятный баг, мешающий корректно решить данную задачу
ultrajson/ultrajson#224

byaka added a commit that referenced this issue May 25, 2016
@byaka
New dev version, prepare for last major release
7a46137 
Added:
 -Added gevent-compatible implementation of raw_input(). This closes #105 issue.
 -Added callAsync() - ability for executing functions without blocking server. This closes #102 issue.
 -Added support of compressed requests. Also implemented auto-using compression in parallelWithSocket exec-backend. This closes #101 issue.
 -Added persistent mode for parallelWithSocket.wait(). This closes #79 issue.
Fixed:
 -Added auto-switch for long-type in ujson, but not solve this. See #78 issue.
 -Fixed perfomance problems with _controlGC(). This closes #106 issue.
 -Rewrited parallelWithSocket exec-backend, now it use socket class, provided by server. This closes #104 issue.
 -Many other small fixes and improvements.
@byaka
Copy link
Owner Author

byaka commented May 25, 2016

Поддержка реализована, но упомянутый выше баг никуда не делся и мешает корректной работе фикса.

@byaka
Copy link
Owner Author

byaka commented Feb 4, 2017

мейнтейнеры ujson закрыли ultrajson/ultrajson#224
нужно еще раз проверить работу фикса и можно закрывать.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@byaka byaka

Labels
high-priority testing
Projects
None yet
Milestone
Last major release
Development

No branches or pull requests
1 participant
@byaka