mjpearson / Pandra

Cassandra abstraction layer and keyspace scaffolder for PHP developers --- ABANDONED.
GNU Lesser General Public License v3.0
93 stars 11 forks source link

"auto" is a vague name for a method #35

Closed jbellis closed 14 years ago

jbellis commented 14 years ago

initialize_all_connections would be more descriptive, although somewhat verbose...

helped someone troubleshoot his app today where he was calling auto() for each request, resulting in a ton of connection overhead :)

BTW, good to see activity on a PHP client, Cassandra needs a standard there like Hector is for Java. Thanks for working on this.

mjpearson commented 14 years ago

thanks for the feedback - fair call and I'll deprecate it out in the next release for something more reasonable. Hope the support request didn't cause too many headaches, it was a poor implementation on my behalf with sockets not timing out (related to https://issues.apache.org/jira/browse/THRIFT-347 i think), but look to have a workaround in place.

thanks, and cheers for your work on Cassandra - its spawned a fun ecosystem.