opened
HyukjinKwon
username_0
### What changes were proposed in this pull request?

This PR proposes to clean up `UnivocityParser`. 

### Why are the changes needed?

It will slightly improve the performance since we don't do unnecessary computation for Array concatenations/creation.

### Does this PR introduce any user-facing change?

No.

### How was this patch tested?

Manually ran the existing tests.
[SPARK-30530][SQL][FOLLOW-UP] Remove unnecessary codes and fix comments accordingly in UnivocityParser
issue
created
HyukjinKwon
username_0
cc @cloud-fan and @username_1. It's rather minor but wanted to clean up here.
comment
created
HyukjinKwon
username_0
retest this please
comment
created
HyukjinKwon
username_0
retest this please
comment
created
MaxGekk
username_1
Something wrong is going on. This exception shouldn't happen:
```
Caused by: java.lang.IllegalStateException: Number of logging event reached the limit: 100
	at org.apache.spark.SparkFunSuite$LogAppender.append(SparkFunSuite.scala:197)
	at org.apache.log4j.AppenderSkeleton.doAppend(AppenderSkeleton.java:251)
```
It was added by https://github.com/apache/spark/pull/27166, and the limit shouldn't be reachable.
comment
created
MaxGekk
username_1
I propose to assign names to log appenders, and print the names in exception. So, we will know which one wasn't removed.
comment
created
srowen
username_2
Jenkins, retest this please
comment
created
HyukjinKwon
username_0
Merged to master.
comment