Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

add batch events method #20

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

add batch events method #20

wants to merge 3 commits into from

Conversation

ice6
Copy link

@ice6 ice6 commented Dec 16, 2015

ccording to the https://docs.prediction.io/datacollection/eventmodel/#3-batch-events-to-the-eventserver
code tested, the code works in my 0.9.5 environment

@pferrel
Copy link

pferrel commented Dec 16, 2015

@EmergentOrder @mkorolyov
We need to do this for the java SDK too

@pferrel
Copy link

pferrel commented Dec 16, 2015

@ice6 do you know why the build failed for PHP 9.5 failed?

@ice6
Copy link
Author

ice6 commented Dec 17, 2015

@pferrel see https://travis-ci.org/PredictionIO/PredictionIO-PHP-SDK/jobs/97362609
the log shows that the ci env is broken :) not because of the code
(btw. PHP5.5 not PHP9.5)

@dantudor thanks, code refactored :)

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

Successfully merging this pull request may close these issues.

3 participants