Merge pull request #257 from nobodyiam/client-refactor-misc

apollo-client/README.md

 ## I. Prerequisite
 
-### I.I Mandatory Setup
+### I.I Requirements
+
+* Java: 1.7+
+
+### I.II Mandatory Setup
 Apollo client requires `AppId` and `Environment` information available to function properly, so please read the following and configure them properly:
 
 #### 1. AppId
@@ -45,7 +49,7 @@ Currently, `env` allows the following values (case-insensitive):
 * UAT
 * PRO
 
-### I.II Optional Setup
+### I.III Optional Setup
 
 #### Cluster
 

apollo-client/src/main/java/com/ctrip/framework/apollo/internals/ConfigRepository.java

@@ -14,7 +14,7 @@ public interface ConfigRepository {
 
   /**
    * Set the fallback repo for this repository.
-   * @param upstreamConfigRepository the fallback repo
+   * @param upstreamConfigRepository the upstream repo
    */
   public void setUpstreamRepository(ConfigRepository upstreamConfigRepository);
 

apollo-client/src/main/java/com/ctrip/framework/apollo/internals/LocalFileConfigRepository.java

@@ -227,12 +227,12 @@ public class LocalFileConfigRepository extends AbstractConfigRepository
     } catch (IOException ex) {
       ApolloConfigException exception =
           new ApolloConfigException(
-              String.format("Create local config directory %s failed", baseDir), ex);
+              String.format("Create local config directory %s failed", baseDir.getAbsolutePath()), ex);
       Cat.logError(exception);
       transaction.setStatus(exception);
       logger.warn(
           "Unable to create local config cache directory {}, reason: {}. Will not able to cache config file.",
-          baseDir, ExceptionUtil.getDetailMessage(ex));
+          baseDir.getAbsolutePath(), ExceptionUtil.getDetailMessage(ex));
     } finally {
       transaction.complete();
     }