[Go][Parquet] Expressions in debug assert parameters got executed, casuing performance issues

[starpact]

Describe the enhancement requested

string concatenation got executed here:

debug.Assert(v <= maxDefLevel, "columnwriter: invalid definition level "+ strconv.Itoa(int(v))+" for column "+w.descr.Path())

missing macro is painful lol, might put this inside a if debug.Enabled() condition block or just remove it?

Component(s)

Parquet

[zeroshade]

interesting, I guess my assumption that it would get optimized out was incorrect. Good catch! Would you be willing to create a PR for this?

[zeroshade]

This issue has been resolved by the merging of PR #708. Closing!