Basic support to Postgresql

[dereuromark]

Basic support to Postgresql

[pedro-stanaka]

@dereuromark After a more thorough study of the class I saw a big number of queries that will have to be rewritten to work on PostgreSQL. I am working on it on my fork, but right now I don't know if it is a good idea to work with switches based on Driver type.

I was thinking of just issuing a very basic query and then do some of the hard work on the application using features like MapReduce of the ORM, but any better ideas are appreciated.

[dereuromark]

Would be nice to have travis CI for the queue. So far it is only tests on MySQL.

Regarding the query, we can also implement a switch based on the type if we can't figure out a good solution.
Usually @lorenzo is the master of ORM :P Especially for this kind of query.

[dereuromark]

I added it to the test matrix for now as allow_failure: https://travis-ci.org/dereuromark/cakephp-queue/jobs/169676171
There might be other small issues to be fixed.

[dereuromark]

Any update here? Maybe someone has the time and can do the postgres support?

[pedro-stanaka]

I actually began to work on it in my fork, but as soon as I realized the amount of queries that would need to be rewritten I put it on hold. It is too much effort to make it work only with postgres, if someone needs to work on other database then the person needs to rewrite everything again. My solution was to create a way simpler method using crontab. Since I am the opener of this issue, if you want @dereuromark you can close it.

[dereuromark]

Well, we can also part by part make the tests pass, and make it work with postgres.
We dont have to make it all work at once.
So feel free to PR.

[dereuromark]

Should only be 2 or 3 queries that really need some adjustments:
https://travis-ci.org/dereuromark/cakephp-queue/jobs/176869139

One is

SELECT (IFNULL(TIMESTAMPDIFF(SECOND, "2016-11-19 01:15:05", notbefore), 0)) AS `age`, QueuedTasks.id AS `QueuedTasks__id`, ...
FROM queued_tasks QueuedTasks 
WHERE ((completed) IS NULL AND (jobtype = 'task1' AND ((notbefore < '2016-11-19 01:15:05' OR (notbefore) IS NULL) AND (fetched < '2016-11-19 01:13:25' OR (fetched) IS NULL)) AND failed < 3)) 
ORDER BY priority ASC, age ASC, id ASC LIMIT 1

[charlieboygis]

Hi, very smart lib.
I made an attempt to have PG support, you can look at my forked version here

[dereuromark]

Nice! You can make a PR once it is ready to be merged. Let me know if you need any help.

[dereuromark]

@charlieboygis Any update here? Can you please also check the latest PR regarding the duplicate issue for PG support?

[dereuromark]

Currently tests are not failing.
Does anyone want to take another stab here to fully integrate PostgreSQL?