CSVPrinter now uses an internal lock instead of synchronized methods

Author: garydgregory

src/main/java/org/apache/commons/csv/CSVPrinter.java

@@ -204,13 +204,25 @@ public long getRecordCount() {
     public void print(final Object value) throws IOException {
         lock.lock();
         try {
-            format.print(value, appendable, newRecord);
-            newRecord = false;
+            printRaw(value);
         } finally {
             lock.unlock();
         }
     }
 
+    /**
+     * Prints the string as the next value on the line. The value will be escaped or encapsulated as needed.
+     *
+     * @param value
+     *            value to be output.
+     * @throws IOException
+     *             If an I/O error occurs
+     */
+    private void printRaw(final Object value) throws IOException {
+        format.print(value, appendable, newRecord);
+        newRecord = false;
+    }
+
     /**
      * Prints a comment on a new line among the delimiter-separated values.
      *
@@ -352,17 +364,17 @@ public void printRecord(final Object... values) throws IOException {
      * separator to the output after printing the record, so there is no need to call {@link #println()}.
      * </p>
      *
-     * @param values
+     * @param stream
      *            values to output.
      * @throws IOException
      *             If an I/O error occurs
      * @since 1.10.0
      */
     @SuppressWarnings("resource") // caller closes.
-    public void printRecord(final Stream<?> values) throws IOException {
+    public void printRecord(final Stream<?> stream) throws IOException {
         lock.lock();
         try {
-            IOStream.adapt(values).forEachOrdered(this::print);
+            IOStream.adapt(stream).forEachOrdered(stream.isParallel() ? this::printRaw : this::print);
             endOfRecord();
         } finally {
             lock.unlock();