Core : Strip trailing slash from tableLocation in LocationProvider

[singhpk234]

Closes #4582

This change attempts to fix :

1. In locationsFor we don't sanitize the location (i.e strip the trailing slash), this attempts to do this stripping
2. The warehouse location for hiveCatalog is also missing this handling rest all catalogs such as Hadoop / Glue have these handling
3. Had to refactor a bit, slash stripping logic was duplicated in all catalogs.
   Note : nessie Catalog also doesn't have this handling, will do the same in next iteration of this pr if experts from nessie suggests this is required (appologies I am new to nessie)
4. Fixes a minor typo.

cc @szehon-ho , @RussellSpitzer , @jackye1995 , @flyrain

[rdblue]

Thanks, @singhpk234! Looks mostly good, but I think we should use just one stripTrailingSlash method.

[rdblue]

Thanks, @singhpk234!