aionnetwork / AVM

Enabling Java code to run in a blockchain environment
https://theoan.com/
MIT License
49 stars 25 forks source link

[CLOSED] Aion collection framework enhancements #238

Closed aionbot closed 5 years ago

aionbot commented 5 years ago

Issue created by yulongaion (on Tuesday Sep 04, 2018 at 20:58 GMT)

aionbot commented 5 years ago

Comment by JunhanHu-aion (on Wednesday Sep 05, 2018 at 20:38 GMT)

237 Prototype hashmap

In progress

217 Add missing exceptions

Done

aionbot commented 5 years ago

Comment by JunhanHu-aion (on Friday Sep 07, 2018 at 17:56 GMT)

Implement "view" based iterator Done with ba99ee8 4ac3083

aionbot commented 5 years ago

Comment by JunhanHu-aion (on Monday Sep 10, 2018 at 13:29 GMT)

After research, I think our current B tree insertion is good enough. "Eager" B tree insertion will result in around 3x of serilization cost and only benefit sequential order insertion. With random order insertion, our current insertion implementation performs better in term of energy.

However, i think there is something can be done for delete opeartion. Currently, each nodes (leaf and internal) are holding a pointer to its parent. Deletion operation relying on parent pointer to find common ancestor (we call it anchor). This parent pointer is not really nessary since we can solve the problem by introducing extra recursion parameters. This optimazion will result in an improvement of memory footprint (saving logn * 8 bytes).

aionbot commented 5 years ago

Comment by JunhanHu-aion (on Monday Sep 10, 2018 at 13:58 GMT)

Supporting java.util.stream is a tricky question. The following compents are identified

I suggest we push it to further release since

aionbot commented 5 years ago

Comment by JunhanHu-aion (on Tuesday Oct 30, 2018 at 13:49 GMT)

Close this since #237 and #217 are closed