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

No spam sub function #249

Merged
merged 16 commits into from
Mar 23, 2017
Merged

No spam sub function #249

merged 16 commits into from
Mar 23, 2017

Conversation

elmato
Copy link
Contributor

@elmato elmato commented Mar 22, 2017

moved to db_notify.cpp

  • get_impacted_account_visitor
  • operation_get_impacted_accounts
  • transaction_get_impacted_accounts
  • get_relevant_accounts
  • database::notify_changed_objects

elmato and others added 16 commits March 22, 2017 20:14
@elmato
Restore bloom filter usage: Check if the object changed/removed was p…
342e330 
…reviously subscribed (read) before sending updates back to ws clients
@elmato
set_subscribe_callback: always initialize bloom filter
0bc0513
@elmato
remove call to notify_changed_objects in _push_transaction
823beb7
@elmato
split notifications (notify_changed_objects) in three signals: new_ob…
02b2672 
…jects, changed_objects, removed_objects
@elmato
allow to apply_block when exception in database signal handler
b2b895a
@elmato
add get_market() function to call_order_object
29c636f
@elmato
handle new database_object signals, refactor
bfa600c
@elmato
Replace clear_filter parameter with notify_remove_create
6f1d8b5
@elmato
Add the array of ids (that are being removed) to the removed_objects …
0ecdc90 
…signal
@elmato
Handle new,changed and removed events from the database_object in one…
31f322c 
… generic function
@elmato
Fix handling of events from database_object in debug_witness_plugin
030577f
@oxarbitrage @elmato
make less false positives( no false positive given in tests with this…
73abb80 
… setting)
@elmato
Include information of impacted accounts relative to the objects crea…
6d3fbee 
…ted/changed/removed. Only extract the impacted accounts information if there are handlers defined in order prevent useless cpu usage on nodes not exposing the rpc api
@elmato
Keep track of subscribed accounts and check for events that reference…
aab5fdd 
… them
@elmato
debug_witness: Handle new params in changed/remove signals from database
cff792a
@elmato
Move notification related functions from db_block.cpp to db_notify.cpp
3eb160b
@vikramrajkumar vikramrajkumar mentioned this pull request Mar 23, 2017
Closed
@vikramrajkumar vikramrajkumar merged commit 8e0b16a into bitshares:master Mar 23, 2017
@vikramrajkumar
Copy link
Contributor

Resolves #231 for reference

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
@elmato @vikramrajkumar @oxarbitrage