Why not take advantage of Keep-Alive? about firebase-php HOT 10 CLOSED

Yes, please. It's really bad to use Firebase in this way.

from firebase-php.

it's relatively easy to implement, please feel free to fork the repository and send a pull request if you want to have this functionality immediately. otherwise i'll examine the best possible implementation soon. thank you for your suggestion.

from firebase-php.

@ktamas77 Too busy with midterms 👎

Waiting for your solution :) Thank you for fast response

from firebase-php.

Just opened a pull request that adds Keep-Alive support. Please test. Thanks.

from firebase-php.

I added your method + fsocks multi set method. #16

And good for news who needs amazing write speeds !

• Added parallel curl with shell_exec; only works in linux ( i guess ).
  But you have no control after setting data, no callbacks no tracing etc.

Some stats (on digitalocean vps - ubuntu 12.04 )

• [default] 50 set request with set method [ it using cURL ]=> 30sec
• [mine]50 set requests with set_multi method [ it using fsocks ] => 12 sec
• [mine]50 set requests with set_fast method [ it using cURL with shell exec /
  parallel ] => 0.4 sec

from firebase-php.

Amazing speed is great and all, but I'm not sure whether it should be the job of an API client library to handle parallelization through platform-specific shell commands. Parallelization should be the job of a different library, of which PHP already has plenty.

Not to mention this opens the door to all sorts of nasty security bugs. (addslashes? seriously?)

The fsockopen version looks cool, though. Could you add Keep-Alive support to that? It seems wasteful to close the socket after every request. As long as you don't need to read the responses, it should be just as fast as, if not faster than, the shell_exec method. Forking a shell is kinda expensive.

Anyway, just my $0.02 as the original opener of this issue. Of course @ktamas77 will have the final say.

from firebase-php.

@kijin haha you're right :)

in push request i mentioned :

Be careful it contains shell_exec method and json_data ( may contains your user inputs ).
so if you are not trusting to data, I suggest you to sanitize it carefully.

What is 0.02$, i don't understand.

(i'm newbie in version controlling and github. Sorry if I made mistake )

from firebase-php.

What is 0.02$, i don't understand.

Haha, it's just an American idiom, "my two cents".

from firebase-php.

@kijin
Dude, i just added keep-alive support for fsocks and removed shell_exec method. #16

You have to try it 👍 IT'S AWESOME !!

from firebase-php.

I agree, it's a nice improvement, please see my comment on #15 and #16

from firebase-php.