ZuInnoTe / hadoopcryptoledger

Hadoop Crypto Ledger - Analyzing CryptoLedgers, such as Bitcoin Blockchain, on Big Data platforms, such as Hadoop/Spark/Flink/Hive
Apache License 2.0
141 stars 51 forks source link

Fixed reading of blocks with sizes between 0x8000-0xFFFF #41

Closed liorregev closed 6 years ago

liorregev commented 6 years ago

As per #40 Fixed by converting signed shorts to unsigned via converting them to ints with the same bytes

jornfranke commented 6 years ago

Hey,

Thank you a lot. I will also include it for rel 1.1.2 this week if the unit tests pass (I can see it only when I login).

Best regards

On 28. Nov 2017, at 14:38, Lior Regev notifications@github.com wrote:

As per #40 Fixed by converting signed shorts to unsigned via converting them to ints with the same bytes

You can view, comment on, or merge this pull request online at:

https://github.com/ZuInnoTe/hadoopcryptoledger/pull/41

Commit Summary

Fixed reading of blocks with sizes between 0x8000-0xFFFF as per ZuInnoTe/hadoopcryptoledger#40 File Changes

M inputformat/src/main/java/org/zuinnote/hadoop/ethereum/format/common/EthereumUtil.java (59) M inputformat/src/test/java/org/zuinnote/hadoop/ethereum/format/mapreduce/EthereumFormatHadoopTest.java (52) A inputformat/src/test/resources/testdata/block403419.bin (0) Patch Links:

https://github.com/ZuInnoTe/hadoopcryptoledger/pull/41.patch https://github.com/ZuInnoTe/hadoopcryptoledger/pull/41.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.