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

Fix recursive read lock #36

Merged
merged 1 commit into from
Aug 1, 2016
Merged

Fix recursive read lock #36

merged 1 commit into from
Aug 1, 2016

Conversation

xichen2020
Copy link

@coveralls
Copy link

coveralls commented Aug 1, 2016 •
edited
Loading

Coverage Status

Coverage increased (+0.5%) to 80.838% when pulling 246ada1 on xichen-fix-recursive-rlock into 7d663c8 on master.

@robskillington
Copy link
Collaborator

LGTM

@xichen2020 xichen2020 merged commit 3fcc173 into master Aug 1, 2016
@xichen2020 xichen2020 deleted the xichen-fix-recursive-rlock branch August 1, 2016 16:00
prateek pushed a commit that referenced this pull request May 11, 2018
@arnikola @prateek
Fix flaky test. Now checks that round robin occurs, rather than speci…
feed2a7 
…fic order (#36)
prateek pushed a commit that referenced this pull request May 12, 2018
@arnikola @prateek
Fix flaky test. Now checks that round robin occurs, rather than speci…
fc38be2 
…fic order (#36)
prateek pushed a commit that referenced this pull request May 12, 2018
@arnikola @prateek
Fix flaky test. Now checks that round robin occurs, rather than speci…
5a0ed0c 
…fic order (#36)
prateek pushed a commit that referenced this pull request May 12, 2018
@arnikola @prateek
Fix flaky test. Now checks that round robin occurs, rather than speci…
6e129e5 
…fic order (#36)
prateek pushed a commit that referenced this pull request Oct 9, 2018
@xichen2020 @prateek
Aggregation server replication (#36)
0aedbc4 
* Aggregation server replication

* Fix tests and address feedback

* Better test coverage

* Add /resign and /status endpoints for integration with tooling

* Cleanup config and update m3cluster dependency

* Update lastFlushed properly and discard metrics during follower flushes

* Process campaign status updates before returning from election loop

* Keep ingesting writes after resignation

* Resignation does not stop election

* Update m3metrics dependency

* Address feedback

* Distinguish different types of responses

* Event based election state transition logic

* Fix tests

* Address comments

* More feedback

* Pull in latest m3cluster change and change shard set id to uint32 type

* Address more feedback
prateek pushed a commit that referenced this pull request Oct 9, 2018
@xichen2020 @prateek
Aggregation server replication (#36)
c7b10bf 
* Aggregation server replication

* Fix tests and address feedback

* Better test coverage

* Add /resign and /status endpoints for integration with tooling

* Cleanup config and update m3cluster dependency

* Update lastFlushed properly and discard metrics during follower flushes

* Process campaign status updates before returning from election loop

* Keep ingesting writes after resignation

* Resignation does not stop election

* Update m3metrics dependency

* Address feedback

* Distinguish different types of responses

* Event based election state transition logic

* Fix tests

* Address comments

* More feedback

* Pull in latest m3cluster change and change shard set id to uint32 type

* Address more feedback
cw9 added a commit that referenced this pull request Oct 12, 2018
@cw9
Scan through buffer in batches (#36)
3ba6de4
cw9 added a commit that referenced this pull request Oct 13, 2018
@cw9
Scan through buffer in batches (#36)
36a8273
cw9 added a commit that referenced this pull request Oct 14, 2018
@cw9
Scan through buffer in batches (#36)
49abb97
cw9 added a commit that referenced this pull request Oct 14, 2018
@cw9
Remove json marshaling code of placement snapshot (#36)
2571f81
cw9 pushed a commit that referenced this pull request Oct 14, 2018
@xichen2020
Comments should be sentences and end in period. (#36)
c73bcd1
cw9 pushed a commit that referenced this pull request Oct 16, 2018
@xichen2020
Comments should be sentences and end in period. (#36)
a912745
cw9 pushed a commit that referenced this pull request Oct 16, 2018
@xichen2020 @cw9
Comments should be sentences and end in period. (#36)
ea312e2
cw9 added a commit that referenced this pull request Oct 16, 2018
@cw9
Remove json marshaling code of placement snapshot (#36)
413d9b5
cw9 added a commit that referenced this pull request Oct 16, 2018
@cw9
Remove json marshaling code of placement snapshot (#36)
d7f8234
cw9 added a commit that referenced this pull request Oct 16, 2018
@cw9
Remove json marshaling code of placement snapshot (#36)
f936762
robskillington pushed a commit that referenced this pull request Feb 11, 2019
Pass error handler into auth handler (#36)
f5f44db 
* pass error handler into auth handler

* tests
robskillington pushed a commit that referenced this pull request Feb 25, 2019
Pass error handler into auth handler (#36)
b95d24f 
* pass error handler into auth handler

* tests
robskillington pushed a commit that referenced this pull request Feb 25, 2019
@robskillington
Pass error handler into auth handler (#36)
df78745 
* pass error handler into auth handler

* tests
prateek pushed a commit that referenced this pull request Apr 7, 2019
@xichen2020 @prateek
Move clock options to m3x (#36)
8930be5
prateek pushed a commit that referenced this pull request Apr 8, 2019
@xichen2020 @prateek
Move clock options to m3x (#36)
d6e5da9
@gediminasgu gediminasgu mentioned this pull request Jun 18, 2020
Merged
jnyi pushed a commit to jnyi/m3 that referenced this pull request Mar 23, 2023
@hczhu
Merge pull request m3db#36 from databricks/new-aggregation-rule-filte…
de245a0 
…r-separator

[PLAT-77308] New aggregation rule filter separator
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.
3 participants
@xichen2020 @coveralls @robskillington