Adding development container config (#559)

* Adding VSCode and dev container conifg
* Cleaned up typos

Author: carlosroman

.devcontainer/devcontainer.json

@@ -0,0 +1,38 @@
+// For format details, see https://aka.ms/devcontainer.json. For config options, see the
+// README at: https://github.com/devcontainers/templates/tree/main/src/java
+{
+	"name": "Java",
+	"image": "mcr.microsoft.com/devcontainers/java:8",
+
+	"features": {
+		"ghcr.io/devcontainers/features/java:1": {
+			"version": "none",
+			"installMaven": "true",
+			"mavenVersion": "3.8.6",
+			"installGradle": "false"
+		},
+		"ghcr.io/devcontainers/features/docker-in-docker:2": {  // Needed for Testcontainers for Java
+			"disableIp6tables": true
+		}
+	},
+
+	// Configure tool-specific properties.
+	"customizations": {
+		// Configure properties specific to VS Code.
+		"vscode": {
+			"settings": {},
+			"extensions": [
+				"streetsidesoftware.code-spell-checker"
+			]
+		}
+	}
+
+	// Use 'forwardPorts' to make a list of ports inside the container available locally.
+	// "forwardPorts": [],
+
+	// Use 'postCreateCommand' to run commands after the container is created.
+	// "postCreateCommand": "java -version",
+
+	// Uncomment to connect as root instead. More info: https://aka.ms/dev-containers-non-root.
+	// "remoteUser": "root"
+}

.gitignore

@@ -10,7 +10,6 @@ target/*
 .settings/
 /target
 .idea
-.vscode
 *.iml
 *.ucls
 
@@ -19,5 +18,5 @@ target/*
 # jenv
 .java_version
 .java-version
-
+!.vscode
 .env

.vscode/settings.json

@@ -0,0 +1,38 @@
+{
+  "java.configuration.updateBuildConfiguration": "disabled",
+  "java.format.settings.url": "style.xml",
+  "java.debug.settings.onBuildFailureProceed": true,
+  "cSpell.words": [
+    "adoptopenjdk",
+    "barbarfoo",
+    "barbarfoobar",
+    "confd",
+    "confs",
+    "fofofofofo",
+    "foofoo",
+    "hamcrest",
+    "jasperserver",
+    "javadocs",
+    "jmxchecknoprefix",
+    "jmxfetch",
+    "jmxremote",
+    "jmxrmi",
+    "jmxtree",
+    "jndi",
+    "JVMS",
+    "keyspace",
+    "mbean",
+    "minibuff",
+    "NADA",
+    "nonblocking",
+    "openj9",
+    "openmbean",
+    "pqrst",
+    "reinit",
+    "statsd",
+    "temurin",
+    "thisis",
+    "thisiscounter",
+    "uripath"
+  ],
+}

src/main/java/org/datadog/jmxfetch/AttachApiConnection.java

@@ -29,7 +29,7 @@ private JMXServiceURL getAddress(Map<String, Object> connectionParams)
         try {
             address = new JMXServiceURL(getJmxUrlForProcessRegex(processRegex));
         } catch (com.sun.tools.attach.AttachNotSupportedException e) {
-            throw new IOException("Unnable to attach to process regex:  " + processRegex, e);
+            throw new IOException("Unable to attach to process regex:  " + processRegex, e);
         }
         return address;
     }

src/main/java/org/datadog/jmxfetch/Instance.java

@@ -276,7 +276,7 @@ private InstanceTelemetry registerTelemetryBean(InstanceTelemetry bean) {
 
         try {
             mbs.registerMBean(bean,instanceTelemetryBeanName);
-            log.debug("Succesfully registered jmx bean for instance {} with ObjectName = {}",
+            log.debug("Successfully registered jmx bean for instance {} with ObjectName = {}",
                 this.getName(), instanceTelemetryBeanName);
         } catch (InstanceAlreadyExistsException
          | MBeanRegistrationException
@@ -534,7 +534,7 @@ public List<Metric> getMetrics() throws IOException {
         return metrics;
     }
 
-    /** Returns whather or not the given period has elapsed since reference time. */
+    /** Returns whether or not the given period has elapsed since reference time. */
     public boolean isPeriodDue(long refTime, Integer refPeriod) {
         if ((System.currentTimeMillis() - refTime) / 1000 < refPeriod) {
             return false;
@@ -543,7 +543,7 @@ public boolean isPeriodDue(long refTime, Integer refPeriod) {
         }
     }
 
-    /** Returns whather or not its time to collect metrics for the instance. */
+    /** Returns whether or not its time to collect metrics for the instance. */
     public boolean timeToCollect() {
         if (this.minCollectionPeriod == null) {
             return true;
@@ -870,7 +870,7 @@ public void cleanUp() {
     }
 
     /**
-     * Asynchronoush cleanup of instance, including connection.
+     * Asynchronous cleanup of instance, including connection.
      * */
     public synchronized void cleanUpAsync() {
         cleanupTelemetryBean();

src/main/java/org/datadog/jmxfetch/JmxAttribute.java

@@ -196,7 +196,7 @@ private List<String> getBeanParametersList(
     }
 
     /**
-     * Wrapper for javax.management.ObjectName.unqoute that removes quotes from the bean parameter
+     * Wrapper for javax.management.ObjectName.unquote that removes quotes from the bean parameter
      * value if possible. If not, it hits the catch block and returns the original parameter.
      */
     private String unquote(String beanParameter) {