use activerecord-imports "import"-method

[aiomaster]

See https://github.com/sunitparekh/data-anonymization/issues/66 I know that there are tests missing for the bulk_processing, cause we would need postgresql or mysql in the tests plus the activerecord-import gem (as development_dependency?) and I don't know how a good setup of these things should be done. I tested the code with my production databases and it works very well. If ::ActiveRecord::Import is not defined or bulk_processing is explicitly turned off by setting bulk_process false at the table definition than nothing changes.

[coveralls]

Coverage decreased (-0.6%) to 93.245% when pulling eee4c5e8ec7213e2373efac23144a6be7034d9f8 on webit-de:master into 6ccfda43e077e32ec153ff3cf916d4622b146562 on sunitparekh:master.

[coveralls]

Coverage decreased (-0.6%) to 93.245% when pulling eee4c5e8ec7213e2373efac23144a6be7034d9f8 on webit-de:master into 6ccfda43e077e32ec153ff3cf916d4622b146562 on sunitparekh:master.

[coveralls]

Coverage decreased (-0.6%) to 93.245% when pulling eee4c5e8ec7213e2373efac23144a6be7034d9f8 on webit-de:master into 6ccfda43e077e32ec153ff3cf916d4622b146562 on sunitparekh:master.

[coveralls]

Coverage decreased (-0.6%) to 93.245% when pulling eee4c5e8ec7213e2373efac23144a6be7034d9f8 on webit-de:master into 6ccfda43e077e32ec153ff3cf916d4622b146562 on sunitparekh:master.

[wasabigeek]

+1

[aiomaster]

Hey @sunitparekh,

is there anything I can do, to get this stuff in?

Thanks!

[sunitparekh]

send pull request

On Mon, Oct 28, 2019 at 11:11 PM Christoph Wagner [email protected] wrote:

Hey @sunitparekh https://github.com/sunitparekh,

is there anything I can do, to get this stuff in?

Thanks!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sunitparekh/data-anonymization/pull/67?email_source=notifications&email_token=AABLNOQX3BUOSML4XDIZK63QQ4P5DA5CNFSM4FK7DMOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECNYVDY#issuecomment-547064463, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABLNOQN2ANRJENOLRE6KMDQQ4P5DANCNFSM4FK7DMOA .

-- thanks & regards, Sunit Parekh +91 99237 00662 [email protected]

[aiomaster]

send pull request

Hi @sunitparekh,

I thought this is a pull request or did I miss something?

Thanks

[sunitparekh]

let me rethink about bulk_import.... I need more time to write it with unit tests...

meanwhile if you are looking at more performant library of data-anon, please give it a try to https://github.com/dataanon/data-anon (Kotiln/Java version with high performant and batch support)

[stevereinke]

Hi there. I would like to use this feature. Is the any progress on this issue? Kind regards