Parquet: Release the compressor when calling close in ParquetWriter

[hmit]

No description provided.

[hmit]

I discovered this after switching to ParquetWriter from iceberg project.
Without this, we see this log message when using Brotli compression to write data files to iceberg.

[jzhuge]

The commit message should have prefix "Parquet: " instead of "Core: ".

[kbendick]

+1. Good catch @hmit!

I tried to find a place where this could be tested, but there didn't seem to be anywhere that was already testing whether or not things were closed so I think it's fine without it.

[hmit]

+1. Good catch @hmit!

I tried to find a place where this could be tested, but there didn't seem to be anywhere that was already testing whether or not things were closed so I think it's fine without it.

Yeah, I didn't find a good unit-test to add this to. However, we did test it by deploying (with brotli compression) and it seems to do the trick.

[jzhuge]

@kbendick Could you please merge this PR?

[szehon-ho]

Thanks @hmit , and @kbendick @jzhuge for review

[hmit]

Thanks all for the quick review and merge!

[jzhuge]

Thanks @hmit for discovering the issue and providing a fix. Thanks @kbendick for review. Thanks @szehon-ho for review and merge!

[kbendick]

@kbendick Could you please merge this PR?

Sorry I couldn't merge this. I don't have that permission yet (hopefully soon). But glad this is in and appreciate the contributions 👍