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

Only one F remains. #6

Merged

Conversation

sergey-nagaytsev
Copy link
Contributor

Please review my addition of one-way format function for date. I feel it's a bit smelly, but can't come up with any other idea. What is documented / standartized way of dealing with dates and timezones in your experience, workshop etc ?

Why did you comment out database fixture creation/cleanup ? Why put in code values different from DB script ?

Tests, among other purposes, are intended for user who freshly downloads a package to be capable of quickly examining it, without digging into sysadmin matters, figuring out how to configure database etc.

Running every test with known state is crucial. No permanent test data. No dependency on running order. I researched DBunit, it can prepare little datasets for every test, but it itself needs Cassandra adapter: sebastianbergmann/dbunit#154

Do you have remote Cassandra with permanent test data ? Why not local ? Disabled system() function ?

@balalaikaboy
Copy link
Owner

I apologize for mistakenly committed the CP4DBaseTestCase file, which disabled local DB creation + cleanup. I reinstalled my PC and connected remotely to my Cassandra system on AWS. I totally agree with you on the test data.
Also, sorry for the late reply. I have just come back from a trip where Internet access was not available :)

balalaikaboy added a commit that referenced this pull request Mar 16, 2015
@balalaikaboy balalaikaboy merged commit 3c539a7 into balalaikaboy:master Mar 16, 2015
@sergey-nagaytsev
Copy link
Contributor Author

ok, got it !

P.S. I have a dream of Cassandra becoming as common as MySQL for PHP, this will make great shift - anyone can write website standing under load of all mankind. This will take lots of networking value improvements - good Doctrine adapter will improve adoption and attracted commitment to pdo_cassandra etc.

My nearest goal is Symfony app with FOSUserBundle and it's companions (authorization etc) work on Cassandra, or even a mock social network (user+groups+friends+messages+blogs+comments) so others can bend it towards more focused community, business software etc.

@balalaikaboy
Copy link
Owner

The last test now passes, please verify. I simply changed the signature of the function into convertToDatabaseValue($value, AbstractPlatform $platform). The reason is PHP does not support method overloading.

I just created this PHP package for fun, hoping someone out here may find it useful. I do use Cassandra extensively for Java projects, where I need to handle lots of writes concurrently. The communication is done via Spring Data Cassandra. Typically I use Cassandra in a mixed environment: MySQL+Cassandra+Solr. The search engine is needed to handle queries across shards and spatial data.

Regarding the CI, I don't know if that is even possible, given the many dependencies and compilation complexity. I ran into much trouble to get it up and running on my Mac, including the manual editing of cassandra_driver.cpp and TBinaryProtocol.h.

I too firmly believe that Cassandra will become the natural choice when it comes to choosing a NoSQL solution for projects with heavy loads. I am currently working with MongoDB as well, yet I still prefer Cassandra for all the good things that it has to offer.

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

Successfully merging this pull request may close these issues.

2 participants