Remove unnecessary string converting in readFooter method

The method (String filePath = file.toString()) is always called even filePath is not used(it is only used when an exception is thrown which is rare).  This kind of string conversion should be avoided when it is not used in terms of memory & CPU efficiency. 

https://github.com/apache/parquet-mr/pull/695