...
| Category | Current implementation | Expected improvements | Difficulty level |
|---|---|---|---|
Iroha connector plugin | 1 Call Engine is left as not implemented. (made the commit rejected within the ledger, and thus rejected in the test case. | ||
2 SetSettingValue is left as not implemented. (returned http 405 Error) | |||
| 3 removePeer is left as not implemented because a valid Iroha testnet is required to use this function. | |||
| 4 fetchCommits is left as not implemented. It could be implemented as something similar to Besu connector's WatchBlockV1. | |||
| 5 There is a problem with producing a pending transaction, the code will get stuck and fail the test suite. Peter looked into this issue, but seems like Iroha ledger itself is struggling to generate the pending transaction. (https://jira.hyperledger.org/browse/IR-1010) | |||
| Iroha-Javascript library | 1 Improve the JS library. The JS library gives “undefined” as output. It would be better for us to do the testcases if it outputs tx status and tx hash. (Right now, I used iroha-helper-ts, but it could be further optimized) | ||
| Iroha docker container | 1 The test cases rely on a modified Iroha v1.2.0 all-in-one (AIO) docker image. | Iroha version should be upgraded to improve Iroha ledger's performance. | 1/5 |
2 Embedded Iroha Python SDK inside the Iroha AIO docker image. Although the current healthcheck mechanism works fine, it enlarges the docker image by ~100MB. | The Iroha docker image could be shrinked once the Iroha team introduces the grpc healthcheck/ once the Iroha team make metrics page an option for v1.2.2. | 1/5 | |
| 3 Each Iroha docker container relies on a corresponding Postgres database container to store information. | The Postgres database docker container could be replaced by RocksDB ( needs just one folder - it should be docker volume - to keep data between different runs of image.) | 2/5 | |
| Documentation | 1 For docs/source/support/iroha.md, the test link is not guaranteed to be valid. | Updating the test link to be a valid link once the current Iroha PR gets merged/released. | 0/5 |