[ 
https://issues.apache.org/jira/browse/CASSANDRA-457?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12759654#action_12759654
 ] 

Eric Evans commented on CASSANDRA-457:
--------------------------------------

I'm of the opinion that we should be pretty liberal about adding stuff to 
contrib/, so while I have a few comments regarding this patch, the only one I 
consider to be blocking is the included copy of simplejson. We really should 
avoid embedding third-party libraries whenever possible, and it doesn't seem 
warranted here (it's easy enough to install simplejson separately).

As for the other (non-blocking) comments:

It's always a good idea to use WSGI when creating Python web applications, but 
having chosen to use mod python instead, there are some features you could 
exploit to make this a little less brittle and a little easier to maintain.

For example, instead of mixing html in with the rest of the code, you could use 
PSP (http://www.modpython.org/live/current/doc-html/pyapi-psp.html).

Instead of hard-coding things like host, port, and keyspace, you could make 
them configurable via the apache configuration using PythonOption directives.

Instead of coding up all of your own form handling, you could take advantage of 
mod python's 
(http://www.modpython.org/live/current/doc-html/pyapi-util-fstor.html).


> 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, 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.

Reply via email to