GH-1701: fix double close of ParquetFileWriter

[naude-r]

Rationale for this change

ParquetFileWriter::close can be called multiple times (end -> close and InternalParquetRecordWriter::close via AutoCloseables.uncheckedClose). this causes an exception to be generated on the 2nd close.

What changes are included in this PR?

Make ParquetFileWriter::close safe to be called multiple times.

Are these changes tested?

Yes.

Are there any user-facing changes?

No.

Closes #1701

[Fokko]

@naude-r I think this is a duplicate of #2951

[Fokko]

@naude-r Thanks for raising this PR. Can you elaborate on where you encountered this issue?

[naude-r]

@naude-r I think this is a duplicate of #2951

thank you....did not notice the other PR. this can be closed in favor of that.

[naude-r]

@naude-r Thanks for raising this PR. Can you elaborate on where you encountered this issue?

since upgrading to 1.14 we've noticed this when performing a ParquetWriter::close. only reason for upgrading was due to our security scanner highlighting an issue.

[wgtmac]

Let me close this. Thanks @naude-r and @Fokko!