Should this warning go in Loader#load, where HIVE_METASTORE_URI_PROP is actually used? It seems wrong to duplicate HIVE_METASTORE_URI_PROP in this class and only check if it's null, rather than using it in the case it's not null.