use activerecord-imports "import"-method

sunitparekh / data-anonymization

Want to use production data for testing, data-anonymization can help you.

MIT License

459 stars 92 forks source link

use activerecord-imports "import"-method #67

Open aiomaster opened 6 years ago

aiomaster commented 6 years ago

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 commented 6 years ago

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

coveralls commented 6 years ago

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

coveralls commented 6 years ago

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

coveralls commented 6 years ago

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

wasabigeek commented 5 years ago

aiomaster commented 5 years ago

Hey @sunitparekh,

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

Thanks!

sunitparekh commented 5 years ago

send pull request

On Mon, Oct 28, 2019 at 11:11 PM Christoph Wagner notifications@github.com 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 parekh.sunit@gmail.com

aiomaster commented 5 years ago

send pull request

Hi @sunitparekh,

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

Thanks

sunitparekh commented 5 years ago

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 commented 4 years ago

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