Skip to content

Code cleanup, updated datastore interface, removed unused files#89

Merged
shatterednirvana merged 22 commits intoAppScale:testingfrom
nlake44:db_updated
Nov 19, 2012
Merged

Code cleanup, updated datastore interface, removed unused files#89
shatterednirvana merged 22 commits intoAppScale:testingfrom
nlake44:db_updated

Conversation

@nlake44
Copy link
Copy Markdown
Contributor

@nlake44 nlake44 commented Nov 6, 2012

Cleaned up files in the datastore end. Updated the interface for cassandra, hypertable, and hbase for scalable query support. Removed top level files which were old and not used. Did soft links for rabbit mq files that were in the labs folder of the python AppServer. Addressed comments from a previous code review.

@nlake44
Copy link
Copy Markdown
Contributor Author

nlake44 commented Nov 6, 2012

This includes issue #13

@ghost ghost assigned jovanchohan Nov 6, 2012
Comment thread AppDB/appscale_datastore_batch.py Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this logger be initialized with the name of the class? (appscale_datastore_batch)?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants