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 <[email protected]> Reviewers: Pavan <[email protected]> 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 <[email protected]> Authored: Thu May 19 18:03:23 2016 +0530 Committer: peeyush b <[email protected]> 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);
