go/runtime/txpool: Refactor runtime scheduling queue

[kostko]

• Remove the algorithm field from the runtime descriptor (it is unused).
• Refactor the runtime txpool as it can be simplified now that there is just one "algorithm".

[codecov]

Codecov Report

Merging #4477 (b4de534) into master (c9cecfb) will increase coverage by 0.07%.
The diff coverage is 91.17%.

@@            Coverage Diff             @@
##           master    #4477      +/-   ##
==========================================
+ Coverage   68.79%   68.87%   +0.07%     
==========================================
  Files         423      421       -2     
  Lines       47241    47174      -67     
==========================================
- Hits        32499    32489      -10     
+ Misses      10747    10697      -50     
+ Partials     3995     3988       -7     

Impacted Files	Coverage Δ
go/registry/api/runtime.go	44.51% <ø> (+0.56%)	⬆️
go/registry/tests/tester.go	92.48% <ø> (-0.01%)	⬇️
go/runtime/txpool/txpool.go	83.59% <84.21%> (+3.39%)	⬆️
go/runtime/txpool/check_queue.go	84.61% <100.00%> (ø)
go/runtime/txpool/priority_queue.go	89.94% <100.00%> (ø)
go/consensus/tendermint/apps/beacon/genesis.go	63.63% <0.00%> (-6.07%)	⬇️
go/common/sgx/common.go	66.01% <0.00%> (-3.89%)	⬇️
go/consensus/tendermint/full/services.go	77.69% <0.00%> (-3.85%)	⬇️
...consensus/tendermint/apps/roothash/transactions.go	70.43% <0.00%> (-3.05%)	⬇️
go/common/grpc/grpc.go	83.43% <0.00%> (-2.37%)	⬇️
... and 30 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c795ad3...b4de534. Read the comment docs.

[ptrus]

Just checking, fixgenesis is not needed for this removal, as the old field will just get skipped when deserializing the json, right?

[kostko]

That's right.