Remove extraneous trailing slash in table location

[puchengy]

Creating an Iceberg table from spark-3.2 with iceberg runtime 0.13.1 with location s3://bucket/db/tbl/ will cause underlying files be like s3://bucket/db/tbl//data/xxx. This leads to other system (e.g. Trino) file not found exception as those system will remove the duplicated slash in the string (s3://bucket/db/tbl/data/xxx).

I think the right behavior should be letting iceberg remove extraneous trailing slash in table location. i.e. when setting the table location as s3://bucket/db/tbl/. the trailing / should be removed.

[nicor88]

@rdblue When running on Iceberg 0.14.0 the issue seems fixed for the data folder, but somehow if I have a trailing slash in my location, the trailing slash is kept for the metadata folder.