Remove exception when getting partition values

[brenoarosa]

Why? :open_book:

Code on get_partition_values_from_table was catch a too general exception unnecessarily. It also caused exceptions on the Hive Metastore.

What? :wrench:

Replaces a try-catch statement to an if.

Type of change :file_cabinet:

• [x] Bug fix (non-breaking change which fixes an issue)

How everything was tested? :straight_ruler:

• Unit tests running
• Stage test environment:
  • Returning same correct partition values for parquet partitioned tables
  • Returning same correct partition values for json partitioned tables
  • Returning same results (empty list) for json unpartitioned tables
  • Not raising exceptions on HMS

Checklist :memo:

• [ ] I have added labels to distinguish the type of pull request.
• [x] My code follows the style guidelines of this project (docstrings, type hinting and linter compliance);
• [x] I have performed a self-review of my own code;
• [x] I have made corresponding changes to the documentation;
• [x] I have added tests that prove my fix is effective or that my feature works;
• [x] I have made sure that new and existing unit tests pass locally with my changes;

[sonarcloud[bot]]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

No Coverage information
0.0% Duplication

[LucasMMota]

You can remove the Attention Points warning if it's not relevant in this PR

[brenoarosa]

Nice catch. It LGTM.

Just a doubt, you said in PR's description that "It also caused exceptions on the Hive Metastore.", but it didn't, right? Since the exceptions were being caught by line 384.

This is how we found the issue. While the exception was caught on this client there was a connected unhandled exception on the Hive Mestastore server.