embulk-input-jira 0.2.8 → 0.2.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a67c13f110c81d075bc30d0b79aa5db339431a43
-  data.tar.gz: 968ad554324ad49f97be6f10a63dbb762aa5f440
+  metadata.gz: 871da4fb82630f92834e2b00784c8e7ccd8b06b6
+  data.tar.gz: 96b6ced1eef2d43a1c258b1d985da224506e2b23
 SHA512:
-  metadata.gz: 70811ba56c74a396048b42cc84d06198a4c14d020d05b78ef568d6fbe13a9ac6450577e386438aa909e91fab6d9be659bce9bf73e80655aa9c27101cb1cf1ff9
-  data.tar.gz: a15ceba7b9bda157a4017fd4fa6f4a2a1c08e559328adae2a8d86c069245fed899028ea027cd7cd22f7cc2239011720aeb5abbbcd5f68ab573661b68f0c26604
+  metadata.gz: c9aab03ff69b21824422a998039c2f0d17eae69d85b59c3aed382acbeb6b940f2b1152b54eb3218c7f252d1360ca2f2cbd92288d8b412fe1c724999ea3a438a8
+  data.tar.gz: e7fd609dc219332b64a9c0a67ffb529b0d65b32b439cce67c506ba986ac2b9cfa26c069e84634a146a9b8afbabb58d962ad2f9ea5b1ac7ddbee70be34a653183

data/.travis.yml

@@ -2,6 +2,8 @@ language: java
 
 jdk: oraclejdk8
 
+dist: trusty 
+
 env:
   global:
     secure: "NCkU3f60cn5Gmzq+NwDmKdQSq+ApF/rPqSHPirk1+ZPW9n+H9912meUuJr9qyn03MPdc6wAQinsl3skdPNh1Qz02TI/WZGMiCQTLh3nKIMzaaxx6I+OZ1TV7A4MoJMQ7oYQdXdbvRYXpOVRnfVDxpUC+BDk2T8xmjfm9cht+PGTzM6RxpmSL842hiudkcyxHxgEf66EGpi6h2G2PqYNfQrksqiVMWlLiTO5a2ee+iw9qTMhgKrhEKLCb+dFB56gtKIOpd3MxieLwf3PDcsjM+TSaWo6Bp+mjdA3zGJNz5wbWRVSdrIvKvxmf3eRqAGssBhBDO/LUN3LnI+/T/7J9XZZd7WXNudo/t6HQXhJkUIvOucCxSg8/uRsxrMAmF2zkJOnYGSvN3HNqdzyz86TK4/Xckl+gX51b0uYncHfZYSojn4py1RWPYpX/L2RHbzMn1MJ7B3/VZ21vY97aHhDgNyyAMxPnTAuR1IZicEgA8y8AmZkkK9tWq5wvAOKI91V+W8iBEaO2dZIib+KdJSUUj/Q3TM3i4wX9O/AU7tknsnCApvxQeveELJ4SxfFd/Z2imqmQWM0wIgp1u7Vjguf8kSaxcuQHonZNdwvvYOkbYxHvUqltzlmE1qFC/kQe6o9fS3NuiugHCYVeFs+LXVVW+nl0pWztFYMicUmQq7T781w="

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.2.9 - 2019-11-17
+
+* [enhancement] Remove usage of `ConfigSource#getObjectNode` [#57](https://github.com/treasure-data/embulk-input-jira/pull/57)
+
 ## 0.2.8 - 2019-04-08
 
 * [fixed] Process is hang forever when validate task [#56](https://github.com/treasure-data/embulk-input-jira/pull/56)

data/README.md

@@ -9,7 +9,7 @@ embulk-input-jira is the Embulk input plugin for [JIRA](https://www.atlassian.co
 
 ## Overview
 
-Required Embulk version >= 0.8.7
+Required Embulk version >= 0.9.20
 
 * **Plugin type**: input
 * **Resume supported**: no

data/build.gradle

@@ -14,21 +14,21 @@ configurations {
     provided
 }
 
-version = "0.2.8"
+version = "0.2.9"
 
 sourceCompatibility = 1.8
 targetCompatibility = 1.8
 
 dependencies {
-    compile  "org.embulk:embulk-core:0.9.11"
-    provided "org.embulk:embulk-core:0.9.11"
+    compile  "org.embulk:embulk-core:0.9.20"
+    provided "org.embulk:embulk-core:0.9.20"
     compile group: 'org.apache.httpcomponents', name: 'httpclient', version: '4.5.6'
     compile group: 'org.glassfish.jersey.core', name: 'jersey-client', version: '2.27'
     compile group: 'com.google.code.gson', name: 'gson', version: '2.8.5'
     testCompile "junit:junit:4.+"
     testCompile "org.mockito:mockito-core:2.+"
-    testCompile "org.embulk:embulk-test:0.9.11"
-    testCompile "org.embulk:embulk-core:0.9.11:tests"
+    testCompile "org.embulk:embulk-test:0.9.20"
+    testCompile "org.embulk:embulk-core:0.9.20:tests"
 }
 
 task classpath(type: Copy, dependsOn: ["jar"]) {

data/src/main/java/org/embulk/input/jira/JiraInputPlugin.java

@@ -162,7 +162,7 @@ public class JiraInputPlugin
         Buffer sample = Buffer.copyOf(createSamples(issues, getUniqueAttributes(issues)).toString().getBytes());
         JsonNode columns = Exec.getInjector().getInstance(GuessExecutor.class)
                                 .guessParserConfig(sample, Exec.newConfigSource(), createGuessConfig())
-                                .getObjectNode().get("columns");
+                                .get(JsonNode.class, "columns");
         return Exec.newConfigDiff().set("columns", columns);
     }
 

data/src/main/java/org/embulk/input/jira/util/JiraUtil.java

@@ -9,6 +9,7 @@ import org.apache.http.client.methods.HttpGet;
 import org.apache.http.impl.client.CloseableHttpClient;
 import org.apache.http.impl.client.HttpClientBuilder;
 import org.embulk.config.ConfigException;
+import org.embulk.config.ConfigSource;
 import org.embulk.input.jira.Issue;
 import org.embulk.input.jira.JiraInputPlugin.PluginTask;
 import org.embulk.spi.Column;
@@ -37,32 +38,32 @@ public final class JiraUtil
 {
     private JiraUtil() {}
 
-    public static int calculateTotalPage(int totalCount, int resultPerPage)
+    public static int calculateTotalPage(final int totalCount, final int resultPerPage)
     {
         return (int) Math.ceil((double) totalCount / resultPerPage);
     }
 
-    public static String buildPermissionUrl(String url)
+    public static String buildPermissionUrl(final String url)
     {
         return UriBuilder.fromUri(url).path(CREDENTIAL_URI_PATH).build().toString();
     }
 
-    public static String buildSearchUrl(String url)
+    public static String buildSearchUrl(final String url)
     {
         return UriBuilder.fromUri(url).path(SEARCH_URI_PATH).build().toString();
     }
 
     public static void validateTaskConfig(final PluginTask task)
     {
-        String username = task.getUsername();
+        final String username = task.getUsername();
         if (isNullOrEmpty(username)) {
             throw new ConfigException("Username or email could not be empty");
         }
-        String password = task.getPassword();
+        final String password = task.getPassword();
         if (isNullOrEmpty(password)) {
             throw new ConfigException("Password could not be empty");
         }
-        String uri = task.getUri();
+        final String uri = task.getUri();
         if (isNullOrEmpty(uri)) {
             throw new ConfigException("JIRA API endpoint could not be empty");
         }
@@ -74,7 +75,7 @@ public final class JiraUtil
                                                                                 .setCookieSpec(CookieSpecs.STANDARD)
                                                                                 .build())
                                             .build()) {
-            HttpGet request = new HttpGet(uri);
+            final HttpGet request = new HttpGet(uri);
             try (CloseableHttpResponse response = client.execute(request)) {
                 response.getStatusLine().getStatusCode();
             }
@@ -82,11 +83,11 @@ public final class JiraUtil
         catch (IOException | IllegalArgumentException e) {
             throw new ConfigException("JIRA API endpoint is incorrect or not available");
         }
-        int retryInitialWaitSec = task.getInitialRetryIntervalMillis();
+        final int retryInitialWaitSec = task.getInitialRetryIntervalMillis();
         if (retryInitialWaitSec < 1) {
             throw new ConfigException("Initial retry delay should be equal or greater than 1");
         }
-        int retryLimit = task.getRetryLimit();
+        final int retryLimit = task.getRetryLimit();
         if (retryLimit < 0 || retryLimit > 10) {
             throw new ConfigException("Retry limit should between 0 and 10");
         }
@@ -96,26 +97,25 @@ public final class JiraUtil
      * For getting the timestamp value of the node
      * Sometime if the parser could not parse the value then return null
      * */
-    private static Timestamp getTimestampValue(PluginTask task, Column column, String value)
+    private static Timestamp getTimestampValue(final PluginTask task, final Column column, final String value)
     {
-        List<ColumnConfig> columnConfigs = task.getColumns().getColumns();
+        final List<ColumnConfig> columnConfigs = task.getColumns().getColumns();
         String pattern = DEFAULT_TIMESTAMP_PATTERN;
-        for (ColumnConfig config : columnConfigs) {
-            if (config.getName().equals(column.getName())
-                    && config.getConfigSource() != null
-                    && config.getConfigSource().getObjectNode() != null
-                    && config.getConfigSource().getObjectNode().get("format") != null
-                    && config.getConfigSource().getObjectNode().get("format").isTextual()) {
-                pattern = config.getConfigSource().getObjectNode().get("format").asText();
+        for (final ColumnConfig columnConfig : columnConfigs) {
+            final ConfigSource columnConfigSource = columnConfig.getConfigSource();
+            if (columnConfig.getName().equals(column.getName())
+                    && columnConfigSource != null
+                    && columnConfigSource.has("format")) {
+                pattern = columnConfigSource.get(String.class, "format");
                 break;
             }
         }
-        TimestampParser parser = TimestampParser.of(pattern, "UTC");
+        final TimestampParser parser = TimestampParser.of(pattern, "UTC");
         Timestamp result = null;
         try {
             result = parser.parse(value);
         }
-        catch (Exception e) {
+        catch (final Exception e) {
         }
         return result;
     }
@@ -124,13 +124,13 @@ public final class JiraUtil
      * For getting the Long value of the node
      * Sometime if error occurs (i.e a JSON value but user modified it as long) then return null
      * */
-    private static Long getLongValue(JsonElement value)
+    private static Long getLongValue(final JsonElement value)
     {
         Long result = null;
         try {
             result = value.getAsLong();
         }
-        catch (Exception e) {
+        catch (final Exception e) {
         }
         return result;
     }
@@ -139,13 +139,13 @@ public final class JiraUtil
      * For getting the Double value of the node
      * Sometime if error occurs (i.e a JSON value but user modified it as double) then return null
      * */
-    private static Double getDoubleValue(JsonElement value)
+    private static Double getDoubleValue(final JsonElement value)
     {
         Double result = null;
         try {
             result = value.getAsDouble();
         }
-        catch (Exception e) {
+        catch (final Exception e) {
         }
         return result;
     }
@@ -154,24 +154,24 @@ public final class JiraUtil
      * For getting the Boolean value of the node
      * Sometime if error occurs (i.e a JSON value but user modified it as boolean) then return null
      * */
-    private static Boolean getBooleanValue(JsonElement value)
+    private static Boolean getBooleanValue(final JsonElement value)
     {
         Boolean result = null;
         try {
             result = value.getAsBoolean();
         }
-        catch (Exception e) {
+        catch (final Exception e) {
         }
         return result;
     }
 
-    public static void addRecord(Issue issue, Schema schema, PluginTask task, PageBuilder pageBuilder)
+    public static void addRecord(final Issue issue, final Schema schema, final PluginTask task, final PageBuilder pageBuilder)
     {
         schema.visitColumns(new ColumnVisitor() {
             @Override
-            public void jsonColumn(Column column)
+            public void jsonColumn(final Column column)
             {
-                JsonElement data = issue.getValue(column.getName());
+                final JsonElement data = issue.getValue(column.getName());
                 if (data.isJsonNull() || data.isJsonPrimitive()) {
                     pageBuilder.setNull(column);
                 }
@@ -181,9 +181,9 @@ public final class JiraUtil
             }
 
             @Override
-            public void stringColumn(Column column)
+            public void stringColumn(final Column column)
             {
-                JsonElement data = issue.getValue(column.getName());
+                final JsonElement data = issue.getValue(column.getName());
                 if (data.isJsonNull()) {
                     pageBuilder.setNull(column);
                 }
@@ -208,14 +208,14 @@ public final class JiraUtil
             }
 
             @Override
-            public void timestampColumn(Column column)
+            public void timestampColumn(final Column column)
             {
-                JsonElement data = issue.getValue(column.getName());
+                final JsonElement data = issue.getValue(column.getName());
                 if (data.isJsonNull() || data.isJsonObject() || data.isJsonArray()) {
                     pageBuilder.setNull(column);
                 }
                 else {
-                    Timestamp value = getTimestampValue(task, column, data.getAsString());
+                    final Timestamp value = getTimestampValue(task, column, data.getAsString());
                     if (value == null) {
                         pageBuilder.setNull(column);
                     }
@@ -226,9 +226,9 @@ public final class JiraUtil
             }
 
             @Override
-            public void booleanColumn(Column column)
+            public void booleanColumn(final Column column)
             {
-                Boolean value = getBooleanValue(issue.getValue(column.getName()));
+                final Boolean value = getBooleanValue(issue.getValue(column.getName()));
                 if (value == null) {
                     pageBuilder.setNull(column);
                 }
@@ -238,9 +238,9 @@ public final class JiraUtil
             }
 
             @Override
-            public void longColumn(Column column)
+            public void longColumn(final Column column)
             {
-                Long value = getLongValue(issue.getValue(column.getName()));
+                final Long value = getLongValue(issue.getValue(column.getName()));
                 if (value == null) {
                     pageBuilder.setNull(column);
                 }
@@ -250,9 +250,9 @@ public final class JiraUtil
             }
 
             @Override
-            public void doubleColumn(Column column)
+            public void doubleColumn(final Column column)
             {
-                Double value = getDoubleValue(issue.getValue(column.getName()));
+                final Double value = getDoubleValue(issue.getValue(column.getName()));
                 if (value == null) {
                     pageBuilder.setNull(column);
                 }

data/src/test/java/org/embulk/input/jira/JiraPluginTestRuntime.java

@@ -19,7 +19,6 @@ import org.embulk.exec.SystemConfigModule;
 import org.embulk.jruby.JRubyScriptingModule;
 import org.embulk.plugin.BuiltinPluginSourceModule;
 import org.embulk.plugin.PluginClassLoaderFactory;
-import org.embulk.plugin.PluginClassLoaderModule;
 import org.embulk.spi.BufferAllocator;
 import org.embulk.spi.Exec;
 import org.embulk.spi.ExecAction;
@@ -37,7 +36,7 @@ public class JiraPluginTestRuntime extends GuiceBinder
 {
     private static ConfigSource getSystemConfig()
     {
-        ObjectNode configNode = JsonNodeFactory.instance.objectNode();
+        final ObjectNode configNode = JsonNodeFactory.instance.objectNode();
         configNode.set("jruby_load_path", JsonNodeFactory.instance.arrayNode().add("lib"));
 
         return new DataSourceImpl(new ModelManager(null, new ObjectMapper()), configNode);
@@ -46,27 +45,26 @@ public class JiraPluginTestRuntime extends GuiceBinder
     public static class TestRuntimeModule implements Module
     {
         @Override
-        public void configure(Binder binder)
+        public void configure(final Binder binder)
         {
-            ConfigSource systemConfig = getSystemConfig();
+            final ConfigSource systemConfig = getSystemConfig();
             new SystemConfigModule(systemConfig).configure(binder);
-            new ExecModule().configure(binder);
+            new ExecModule(systemConfig).configure(binder);
             new ExtensionServiceLoaderModule(systemConfig).configure(binder);
             new BuiltinPluginSourceModule().configure(binder);
             new JRubyScriptingModule(systemConfig).configure(binder);
-            new PluginClassLoaderModule(systemConfig).configure(binder);
             new TestUtilityModule().configure(binder);
             new TestPluginSourceModule().configure(binder);
         }
     }
 
-    private ExecSession exec;
+    private final ExecSession exec;
 
     public JiraPluginTestRuntime()
     {
         super(new TestRuntimeModule());
-        Injector injector = getInjector();
-        ConfigSource execConfig = new DataSourceImpl(injector.getInstance(ModelManager.class));
+        final Injector injector = getInjector();
+        final ConfigSource execConfig = new DataSourceImpl(injector.getInstance(ModelManager.class));
         this.exec = ExecSession.builder(injector).fromExecConfig(execConfig).build();
     }
 
@@ -96,10 +94,11 @@ public class JiraPluginTestRuntime extends GuiceBinder
     }
 
     @Override
-    public Statement apply(Statement base, Description description)
+    public Statement apply(final Statement base, final Description description)
     {
         final Statement superStatement = JiraPluginTestRuntime.super.apply(base, description);
         return new Statement() {
+            @Override
             public void evaluate() throws Throwable
             {
                 try {
@@ -107,13 +106,13 @@ public class JiraPluginTestRuntime extends GuiceBinder
                         try {
                             superStatement.evaluate();
                         }
-                        catch (Throwable ex) {
+                        catch (final Throwable ex) {
                             throw new RuntimeExecutionException(ex);
                         }
                         return null;
                     });
                 }
-                catch (RuntimeException ex) {
+                catch (final RuntimeException ex) {
                     throw ex.getCause();
                 }
                 finally {
@@ -125,7 +124,7 @@ public class JiraPluginTestRuntime extends GuiceBinder
 
     private static class RuntimeExecutionException extends RuntimeException
     {
-        public RuntimeExecutionException(Throwable cause)
+        public RuntimeExecutionException(final Throwable cause)
         {
             super(cause);
         }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: embulk-input-jira
 version: !ruby/object:Gem::Version
-  version: 0.2.8
+  version: 0.2.9
 platform: ruby
 authors:
 - uu59
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-04-08 00:00:00.000000000 Z
+date: 2019-11-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -56,7 +56,7 @@ files:
 - build.gradle
 - classpath/commons-codec-1.10.jar
 - classpath/commons-logging-1.2.jar
-- classpath/embulk-input-jira-0.2.8.jar
+- classpath/embulk-input-jira-0.2.9.jar
 - classpath/gson-2.8.5.jar
 - classpath/httpclient-4.5.6.jar
 - classpath/httpcore-4.4.10.jar