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

Feat 3996 bzip2 compression #89

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

AkhileshJyotishi
Copy link

Create BZIP2 Compression Class and Unit Test

Description

This pull request introduces a new compression class to support BZIP2 compression and decompression, along with its accompanying unit test. The BZIP2 compression algorithm is now available for use in our storage library, enhancing its capabilities.

Installation Instructions (for Linux-based distributions)

Before using the BZIP2 compression class, you need to install the required dependency. You can do this using the following command on Linux-based distributions:

sudo apt install php-bz2

Testing command

./vendor/bin/phpunit  ./tests/Storage/Compression/Algorithms/BZIP2TEST.php

Attached Screenshot

Screenshot_20231003_214438

closes: appwrite/appwrite#3996

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we rename this file BZIP2Test.php?


$this->assertEquals($this->object->decompress($data), $demo);
}

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add the testCompressDecompressWithLargeText() as seen in other compression tests?

$this->assertEquals($demoSize, 3038056);
$this->assertEquals($dataSize, 2999345);

$this->assertGreaterThan($dataSize, $demoSize);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we need this

$this->assertEquals($demoSize, 599639);
$this->assertEquals($dataSize, 598565);

$this->assertGreaterThan($dataSize, $demoSize);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this is necessary

Copy link
Contributor

@loks0n loks0n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also fix the linter by running composer run format

@AkhileshJyotishi
Copy link
Author

please give me hacktoberfest accepted tag i will update it soon

@AkhileshJyotishi
Copy link
Author

@loks0n

@loks0n
Copy link
Contributor

loks0n commented Oct 31, 2023

Please don't spam. You already have the tag.

@tessamero
Copy link

Hi @AkhileshJyotishi

Thank you for your contribution to Hacktoberfest 2023! We've noticed that your PR is still pending and requires some updates based on our engineering team's feedback.

We would love to see your PR successfully merged and send you the Appwrite swag as a token of appreciation. To remain eligible for the swag, please address the pending suggestions and/or ensure the tests pass by Friday, November 17th. If the PR isn't updated by then, we will unfortunately have to close it due to the end of the Hacktoberfest event.

@gewenyu99
Copy link

Hey there! There were a lot of big PRs during this Hacktoberfest, and we wanted to give everyone ample time to collaborate with our engineering team. If you were able to merge your PRs during October, amazing. If it’s still not merged, don’t worry about it either. Either way, we’ve got your Hacktoberfest swag minted and ready to ship.

Please comment with your Discord username here so we can contact you about your shipping information to deliver your Hacktoberfest swag.

@AkhileshJyotishi
Copy link
Author

@gewenyu99

my username akhilesh_32621

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🗜️ Refine Appwrite Storage with bzip2 Compression
4 participants