AbedElazizShe / LightCompressor

A powerful and easy-to-use video compression library for android uses MediaCodec API.
Apache License 2.0
529 stars 116 forks source link

Unable to Compress multiple videos parallely #110

Closed nikhilrfitplus closed 2 years ago

nikhilrfitplus commented 2 years ago

Thanks for yesterday's commit which is now allowing to pass list of file URIs. But, my core requirement is to get the files compressed in parallel. When i call VideoCompressor.start(.. for the 2nd time, the 1st compression task is getting terminated and calling the successCallBack method, and then the 2nd compression task is starting. Please add a provision to compress the files parallely or allow to create multiple objects/instances of VideoCompressor class

AbedElazizShe commented 2 years ago

Hi @nikhilrfitplus updated in 1.1.1. Please let me know if you have any issues as I only tested it in a small set of devices.

nikhilrfitplus commented 2 years ago

Hi AbedElazizShe

Thanks for the prompt reply & instant fix. It's working now as per my unit test. We have scheduled a QA test tomorrow. Will let you know if we come across any issues.

Regards, Nikhil

On Sun, 23 Jan 2022 at 16:22, Abed Elaziz Shehadeh @.***> wrote:

Hi @nikhilrfitplus https://github.com/nikhilrfitplus updated in 1.1.1. Please let me know if you have any issues as I only tested it in a small set of devices.

— Reply to this email directly, view it on GitHub https://github.com/AbedElazizShe/LightCompressor/issues/110#issuecomment-1019458792, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXNS5GPZHSYE5FMENAMVIZDUXPMYJANCNFSM5MS57XLA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>