You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@falcon.apache.org by pe...@apache.org on 2016/05/19 12:33:42 UTC

falcon git commit: FALCON-1973 Falcon build failure due to checkstyle issue

Repository: falcon
Updated Branches:
  refs/heads/master 6e034074f -> a9119c696


FALCON-1973 Falcon build failure due to checkstyle issue

Please review the following pull request in which Falcon build failure is happening with checkstyle issue from the FALCON-1964 patch.

Author: peeyush b <pb...@hortonworks.com>

Reviewers: Pavan <pa...@gmail.com>

Closes #149 from peeyushb/FALCON-1973


Project: http://git-wip-us.apache.org/repos/asf/falcon/repo
Commit: http://git-wip-us.apache.org/repos/asf/falcon/commit/a9119c69
Tree: http://git-wip-us.apache.org/repos/asf/falcon/tree/a9119c69
Diff: http://git-wip-us.apache.org/repos/asf/falcon/diff/a9119c69

Branch: refs/heads/master
Commit: a9119c6963b7862f5c784a6ffff8d3b1551ca6b5
Parents: 6e03407
Author: Peeyush <pe...@apache.org>
Authored: Thu May 19 18:03:23 2016 +0530
Committer: peeyush b <pb...@hortonworks.com>
Committed: Thu May 19 18:03:23 2016 +0530

----------------------------------------------------------------------
 .../org/apache/falcon/util/ApplicationPropertiesTest.java | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/falcon/blob/a9119c69/common/src/test/java/org/apache/falcon/util/ApplicationPropertiesTest.java
----------------------------------------------------------------------
diff --git a/common/src/test/java/org/apache/falcon/util/ApplicationPropertiesTest.java b/common/src/test/java/org/apache/falcon/util/ApplicationPropertiesTest.java
index c22b6ad..1ab4071 100644
--- a/common/src/test/java/org/apache/falcon/util/ApplicationPropertiesTest.java
+++ b/common/src/test/java/org/apache/falcon/util/ApplicationPropertiesTest.java
@@ -40,14 +40,14 @@ public class ApplicationPropertiesTest {
     private static final String PROPERTY_2 = "property-key-2";
     private static final String JKS_FILE_NAME = "credentials.jks";
 
-    private static final File credDir = new File(".");
+    private static final File CRED_DIR = new File(".");
 
     @AfterClass
     public void tearDown() throws Exception {
         // delete temporary jks files
-        File file = new File(credDir, JKS_FILE_NAME);
+        File file = new File(CRED_DIR, JKS_FILE_NAME);
         file.delete();
-        file = new File(credDir, "." + JKS_FILE_NAME + ".crc");
+        file = new File(CRED_DIR, "." + JKS_FILE_NAME + ".crc");
         file.delete();
     }
 
@@ -57,12 +57,12 @@ public class ApplicationPropertiesTest {
         Assert.assertTrue(CredentialProviderHelper.isProviderAvailable());
 
         // clean credential provider store
-        File file = new File(credDir, JKS_FILE_NAME);
+        File file = new File(CRED_DIR, JKS_FILE_NAME);
         file.delete();
 
         // add alias to hadoop credential provider
         Configuration conf = new Configuration();
-        String providerPath = "jceks://file/" + credDir.getAbsolutePath() + "/" + JKS_FILE_NAME;
+        String providerPath = "jceks://file/" + CRED_DIR.getAbsolutePath() + "/" + JKS_FILE_NAME;
         conf.set(CredentialProviderHelper.CREDENTIAL_PROVIDER_PATH, providerPath);
         CredentialProviderHelper.createCredentialEntry(conf, ALIAS_1, PASSWORD_1);
         CredentialProviderHelper.createCredentialEntry(conf, ALIAS_2, PASSWORD_2);