search

OpenBEL / openbel-framework

An open platform for capturing, integrating, storing, and sharing biological knowledge in and across organizations.
http://openbel.org
Apache License 2.0
22 stars 7 forks source link

Remove support for KAM database encryption #20

Closed JamesMcMahon closed 11 years ago

JamesMcMahon commented 12 years ago

KAM database encryption should be something the user can disable.

This will make debugging easier and remove unneeded complexity in settings where encryption is not appropriate.

nbargnesi commented 12 years ago

We should get an understanding of the time it takes to compile the current large corpus for our current experimental branch.

We're expecting a slight reduction in the time it takes to compile the documents once encryption support is removed.

abargnesi commented 11 years ago

I've removed encryption from the framework. This includes removing encryption api, implementation, packages, references, and the passphrase system configuration variable. Javadoc was also updated where appropriately.

The commons-codec dependency was no longer needed in framework.core so it was moved to framework.ws where it is needed.

_The KAM database schema was not updated but string values in objects.varchar_value / objects_text.textvalue are stored as-is.

I compiled the large corpus before and after. I saw a 2.4% improvement in wall-clock time (~15 second difference).