-
Notifications
You must be signed in to change notification settings - Fork 167
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.
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
Candidate for 5.0.17 release #2067
Conversation
Test for garbage-collecting utxo with invalid ergo tree
Change logging maxHistory to 1 week
Minor refactorings Moved some scorex.core classes into org.ergoplatform To simplify creation of alternative clients for Ergo.
Extract NiPoPow
…edErgoTransaction
#2051 Add limit and offset parameter to /wallet/boxes/unspent
After-#2060 refactoring
# Conflicts: # ergo-core/src/main/scala/org/ergoplatform/http/api/ApiCodecs.scala # src/main/scala/org/ergoplatform/http/api/BlockchainApiRoute.scala # src/main/scala/org/ergoplatform/nodeView/history/extra/ExtraIndexer.scala # src/main/scala/org/ergoplatform/nodeView/history/extra/IndexedErgoTransaction.scala
…nto i2066-indexer-rework
val scanId = ScanId @@ scanIdInt.toShort | ||
val considerUnconfirmed = minConfNum == -1 | ||
withWallet(_.scanUnspentBoxes(scanId, considerUnconfirmed, minHeight, maxHeight).map { | ||
_.filter(boxConfirmationFilter(_, minConfNum, maxConfNum)) | ||
.slice(offset, offset + limit) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't the slicing be pushed down to the data extraction stack?
|
||
val ASC: Direction = 1.asInstanceOf[Direction] | ||
val DESC: Direction = 0.asInstanceOf[Direction] | ||
val INVALID: Direction = (-1).asInstanceOf[Direction] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ccellado, maybe .toByte would be more clean, don't you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed asInstanceOf in e93daf0
Candidate for 5.0.17 reference client release. It contains following PRs: