[
https://issues.apache.org/jira/browse/CASSANDRA-457?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760422#action_12760422
]
Hernan Badenes commented on CASSANDRA-457:
------------------------------------------
Thank you Eric and Michael for your comments. I applied most of your
suggestions:
- used 'try import json; except ...'. We were using simplejson as 'json' is not
available in all systems; but the inclusion on this patch was unintentional.
There is an annotation in the README on how to obtain json or simplejson
libraries, so that should be enough.
- we changed form processing to use FieldStorage. This simplified the code very
nicely.
- also moved the default values to settings you can pass with PythonOptions
through server's mod_python.
- I also cleaned up the code a little -- but it is still written "a la
publisher" way (not PSP). Doing it with PSP is feasible... but I also like how
publisher handles things -- a middle ground would be great, though I am not
sure how to implement this. In any case I don't have such a strong opinion on
using either of the approaches, so if you have it and think that changing it is
a must, please let me know.
I have just uploaded a new version intended to replace the v1 patch.
> web-based Cassandra browser
> ---------------------------
>
> Key: CASSANDRA-457
> URL: https://issues.apache.org/jira/browse/CASSANDRA-457
> Project: Cassandra
> Issue Type: Improvement
> Affects Versions: 0.5
> Reporter: Jun Rao
> Assignee: Jun Rao
> Attachments: issue457.patchv1, issue457.patchv2, screenshot.pdf
>
>
> A web-based interface that simplifies accessing cassandra through thrift.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
