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.

Sign up for GitHub

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

Jump to bottom

Ad server connector remove #13

Closed
wants to merge 44 commits into from
Closed

Ad server connector remove #13

wants to merge 44 commits into from

Conversation

wsourdeau
Copy link
Contributor

No description provided.

Wolfgang Sourdeau and others added 30 commits March 18, 2013 22:58
removed handling of visit events
75a0275
removed doVisit and related functions from PA loop
9975378
Merge remote-tracking branch 'origin/remove-visit-handling' into remo…
e765ece 
…ve-visit-handling

Conflicts:
	core/post_auction/post_auction_loop.cc
	core/post_auction/post_auction_loop.h
reverted excessive changes
5f64a13
restored changes to logger/
e051745
Merge branch 'remove-visit-handling' into pa_loop-events_generalisation
be5d520
invoke finish() on auctions that are ignored in slow mode
b665a8b
Merge branch 'pa_loop-finish_slowmode_auctions' into pa_loop-events_g…
bbbe464 
…eneralisation
Merge branch 'master' into pa_loop-events_generalisation
a8100cc
replaced impression/click methods with equivalent campaint event methods
f0261ee
make sure the number of events is serialized as an int
a6d42a7
removed ifdeffed portions
9f76196
fixed various forms of CampaignEvent serialization, including to the …
8438de8 
…logger and to the agents
fixed retrieval of event label
8611d57
json serialisation of time value put in "timestamp" as before
d8c0196
Merge branch 'master' into pa_loop-events_generalisation
6311ac3
added unit test that ensures the values of AccountSummary are consist…
7150612 
…ent with those of the underlying accounts
soa submodule catchup for redis spin polling fix.
6f9b70a
Added additional timing metrics for the augmentor message flow.
f69669a
removed empty lines
12cf53a
fixed include
cdcd07c
moved PostAuctionEvent in common/
52f9244
Merge branch 'master' into pa_loop-events_generalisation
0610a33
Merge branch 'pa_loop-events_generalisation' into post_auction_event-…
67ebd6a 
…common
removed left-over declarations of injectClick/injectImpressions
7683fb7
updated js binding for RouterStack
655285f
Merge branch 'pa_loop-events_generalisation' into post_auction_event-…
d9826ab 
…common
removed left-over declarations of injectClick/injectImpressions
5b9c468
Merge branch 'pa_loop-events_generalisation' into post_auction_event-…
15b8dac 
…common
added post auction runner
2ae1dd8
@nicolaskruchten
Copy link
Contributor

We should squash the commits so that all tests pass on all commits...

@wsourdeau wsourdeau closed this Apr 3, 2013
@pbeizman pbeizman mentioned this pull request Nov 15, 2013
Closed
oktal pushed a commit that referenced this pull request Apr 10, 2014
Merge pull request #13 from datacratic/unicode-everywhere-2
3b12d65 
Unicode Everywhere
woud420 added a commit that referenced this pull request Nov 5, 2014
@woud420
Merge pull request #13 from datacratic/makefile_changes
2dcc133 
setting fortran compiler to gfortran by default in .mk so we dont have t...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@wsourdeau @nicolaskruchten