Courtesy of @mpchadwick
i'm also wondering there should be pagination used here https://github.com/genecommerce/module-encryption-key-manager/blob/master/Service/ChangeEncryptionKey.php#L56-L60 . I don't think we've seen it yet, but I think fetchPairs wlll attempt to load the entire dataset into memory, so I do think this can run out of memory at a certain point
The custom reencrypt-column command also may need a review.
Courtesy of @mpchadwick
The custom
reencrypt-columncommand also may need a review.