Revert "Hive: close the fileIO client when closing the hive catalog"

[Fokko]

This reverts commit 7e2920a from #10771.

Issue reported in #11783, #11633 and #11582.

I was able to reproduce this on PyIceberg apache/iceberg-python#1323, and while doing a git bisect, this led me to this PR:

➜  iceberg git:(7e2920af4) git bisect bad       
7e2920af400e5d559f8f1742b1043787b0477d74 is the first bad commit
commit 7e2920af400e5d559f8f1742b1043787b0477d74
Author: Hussein Awala <[email protected]>
Date:   Thu Jul 25 19:50:19 2024 +0200

    Hive: close the fileIO client when closing the hive catalog (#10771)
    
    Co-authored-by: Amogh Jahagirdar <[email protected]>
    Co-authored-by: Eduard Tudenhoefner <[email protected]>

 .../java/org/apache/iceberg/hive/HiveCatalog.java  | 33 +++++++++++++++++++++-
 1 file changed, 32 insertions(+), 1 deletion(-)

For now, I suggest reverting it to backport this to 1.7.2.

Closes #11783
Closes #11633

cc @hussein-awala @nastra @amogh-jahagirdar

[hussein-awala]

LGTM for the revert. I will try to find out why he introduced this bug.

Thanks @Fokko!

[amogh-jahagirdar]

Thanks @Fokko! While the original fix intention was to close FileIO, seems like that's not happening correctly given the lifecycle/relation of HiveTableOperations to the HiveCatalog, and it's impacting operations! In the short term, agree with reverting, worst case is some uncleaned FileIOs but it won't impede jobs at least. Long term, will have to figure out how to track/close FileIO at the right time.

[amogh-jahagirdar]

I'll go ahead and merge, thanks for reviewing @hussein-awala @bryanck !

[Fokko]

Thanks for reviewing this @hussein-awala, @bryanck and @amogh-jahagirdar. I fully agree with Amoghs' assessment. Let me cherry-pick this to the 1.7.x branch