Allow the normalize limit to be set via a flag.

[martinmr]

Fixes #3335

---

This change is 

[mangalaman93]

LGTM. Running CI again, tests seem to fail.

[manishrjain]

One comment. Use flag.Int instead of flag.Uint64. After fixing, feel free to merge.

Reviewed 3 of 3 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @martinmr)

---

query/outputnode.go, line 281 at r1 (raw file):

		for _, ca := range child {
			cnt += len(pa) + len(ca)
			if cnt > int(x.Config.NormalizeNodeLimit) {

Can't you use flag.Int instead of using flag.Uint64 then casting it into int?

[martinmr]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @manishrjain)

---

query/outputnode.go, line 281 at r1 (raw file):

Previously, manishrjain (Manish R Jain) wrote…

Can't you use flag.Int instead of using flag.Uint64 then casting it into int?

I kept the flag as a uint to prevent someone from setting it to a negative value but I made NormalizeNodeLimit an int so the casting from uint to int is only done once.