Spent claims #77

Merged
tiger5226 merged 4 commits from spent_claims into master 2018-05-27 15:50:53 +02:00
tiger5226 commented 2018-05-27 04:57:45 +02:00 (Migrated from github.com)

-Added content type to gendb to support elastic search 6.+ comms.
-Fixed bug that causes completionmapper parse error in elastic search. This causes an entire batch to fail. Batches are up to 500 so if 1 fails they all fail. The impact is that many claims were not indexed in elastic search.
-Parameterized elastic search logging to a const.
-Refactored handling of blocked claims.
-Changed deletes to be queued and refactored.
-Added logging.
-Removed commented debug code.
-Added Spent and Expired bid_state handling.

https://github.com/lbryio/lighthouse/pull/77/files

-Added content type to gendb to support elastic search 6.+ comms. -Fixed bug that causes completionmapper parse error in elastic search. This causes an entire batch to fail. Batches are up to 500 so if 1 fails they all fail. The impact is that many claims were not indexed in elastic search. -Parameterized elastic search logging to a const. -Refactored handling of blocked claims. -Changed deletes to be queued and refactored. -Added logging. -Removed commented debug code. -Added Spent and Expired bid_state handling. https://github.com/lbryio/lighthouse/pull/77/files
filipnyquist (Migrated from github.com) approved these changes 2018-05-27 13:36:32 +02:00
filipnyquist (Migrated from github.com) left a comment

Looks good, no problems on my end!

Looks good, no problems on my end!
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lighthouse.js#77
No description provided.