it strictly depends on your needs and how your validator is supposed to work.
traces in general (trace
, traceIfFalse
, traceIfTrue
, traceError
) are known to be one of the causes of larger scripts.
so if your validator has to be used in pair with other data in a transaction it is essential it reduces the script size.
keep in mind that removing traces means you'll likely have a hard time understanding what is wrong if something goes wrong.
one possible solution to keep traces would be to use error codes ( as done for some errors that might happen when using builtin funcitonalities ) so that you can reduce the script size and have a clue of what's going on in your code.