Improve getBlockByNumber performance with lazy-loaded JSON

[wanliqun]

to reduce redundant JSON marshalling and unmarshalling to improve performance

---

This change is 

[boqiu]

Reviewed 2 of 3 files at r1, 4 of 4 files at r2, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @wanliqun)

---

rpc/lazy.go line 8 at r2 (raw file):

)

type lazyDecodedJsonObject[T any] struct {

Better to add necessary comment for performance consideration.

---

rpc/lazy.go line 9 at r2 (raw file):

type lazyDecodedJsonObject[T any] struct {
	cachedBytes json.RawMessage

Why not use []byte type directly? Because the input parameter type of UnmarshalJSON is []byte.

[wanliqun]

Reviewable status: 3 of 4 files reviewed, 2 unresolved discussions (waiting on @boqiu)

---

rpc/lazy.go line 8 at r2 (raw file):

Previously, boqiu (Bo QIU) wrote…

Better to add necessary comment for performance consideration.

Done

---

rpc/lazy.go line 9 at r2 (raw file):

Previously, boqiu (Bo QIU) wrote…

Why not use []byte type directly? Because the input parameter type of UnmarshalJSON is []byte.

I think `json.RawMessage` is okay because it explicitly indicates that the field contains raw JSON data, improving code readability and understanding.

[boqiu]

Reviewed 1 of 1 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @wanliqun)