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.

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

switch to using a SHA256 hash of the conformer string #51

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

greglandrum
Copy link
Contributor

@greglandrum greglandrum commented Apr 3, 2024

this is a simple one to avoid ENORMOUS conformer hashes

Provides a fix to one of the problems #50 was working around.

An existing conformer table can be modified in place to support working with this updated version of the hash:

alter table conformers add column c_hash_2 text;
update conformers set c_hash_2 = substring(sha256(conformer_hash::bytea)::text from 3);
alter table conformers drop column conformer_hash;
alter table conformers rename column c_hash_2 to conformer_hash;

so that we do not end up with ENORMOUS conformer hashes
@greglandrum greglandrum added the enhancement New feature or request label Apr 3, 2024
@greglandrum greglandrum requested a review from brje01 April 3, 2024 09:12
@brje01 brje01 merged commit 6cc9714 into main Apr 3, 2024
1 check passed
@brje01 brje01 deleted the dev/smarter_conformer_hash branch April 3, 2024 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants