Convert more classes

Author: heyams

agent/agent-profiler/agent-diagnostics/src/main/java/com/microsoft/applicationinsights/diagnostics/collection/libos/hardware/MemoryInfo.java

@@ -3,45 +3,64 @@
 
 package com.microsoft.applicationinsights.diagnostics.collection.libos.hardware;
 
-import com.fasterxml.jackson.annotation.JsonCreator;
-import com.fasterxml.jackson.annotation.JsonProperty;
+import com.azure.json.JsonSerializable;
+import com.azure.json.JsonWriter;
+import java.io.IOException;
 
-public class MemoryInfo {
+public class MemoryInfo implements JsonSerializable<MemoryInfo> {
 
-  private final long totalInKb;
+  private long totalInKb;
 
-  private final long freeInKb;
+  private long freeInKb;
 
-  private final long virtualMemoryTotalInKb;
+  private long virtualMemoryTotalInKb;
 
-  private final long virtualMemoryUsedInKb;
-
-  @JsonCreator
-  public MemoryInfo(
-      @JsonProperty("totalInKB") long totalInKb,
-      @JsonProperty("freeInKB") long freeInKb,
-      @JsonProperty("virtualMemoryTotalInKB") long virtualMemoryTotalInKb,
-      @JsonProperty("virtualMemoryUsedInKB") long virtualMemoryUsedInKb) {
-
-    this.totalInKb = totalInKb;
-    this.freeInKb = freeInKb;
-    this.virtualMemoryTotalInKb = virtualMemoryTotalInKb;
-    this.virtualMemoryUsedInKb = virtualMemoryUsedInKb;
-  }
+  private long virtualMemoryUsedInKb;
 
   public long getTotalInKb() {
     return totalInKb;
   }
 
+  public MemoryInfo setTotalInKb(long totalInKb) {
+    this.totalInKb = totalInKb;
+    return this;
+  }
+
   public long getFreeInKb() {
     return freeInKb;
   }
 
+  public MemoryInfo setFreeInKb(long freeInKb) {
+    this.freeInKb = freeInKb;
+    return this;
+  }
+
   public long getVirtualMemoryTotalInKb() {
     return virtualMemoryTotalInKb;
   }
 
+  public MemoryInfo setVirtualMemoryTotalInKb(long virtualMemoryTotalInKb) {
+    this.virtualMemoryTotalInKb = virtualMemoryTotalInKb;
+    return this;
+  }
+
   public long getVirtualMemoryUsedInKb() {
     return virtualMemoryUsedInKb;
   }
+
+  public MemoryInfo setVirtualMemoryUsedInKb(long virtualMemoryUsedInKb) {
+    this.virtualMemoryUsedInKb = virtualMemoryUsedInKb;
+    return this;
+  }
+
+  @Override
+  public JsonWriter toJson(JsonWriter jsonWriter) throws IOException {
+    jsonWriter.writeStartObject();
+    jsonWriter.writeLongField("totalInKb", totalInKb);
+    jsonWriter.writeLongField("freeInKb", freeInKb);
+    jsonWriter.writeLongField("virtualMemoryTotalInKb", virtualMemoryTotalInKb);
+    jsonWriter.writeLongField("virtualMemoryUsedInKb", virtualMemoryUsedInKb);
+    jsonWriter.writeEndObject();
+    return jsonWriter;
+  }
 }

agent/agent-profiler/agent-diagnostics/src/main/java/com/microsoft/applicationinsights/diagnostics/collection/libos/os/linux/LinuxMemoryInfoReader.java

@@ -16,7 +16,12 @@ public class LinuxMemoryInfoReader extends TwoStepProcReader implements MemoryIn
   private int freeInKbIndex = -1;
   private int virtualMemoryTotalInKbIndex = -1;
   private int virtualMemoryUsedInKbIndex = -1;
-  private MemoryInfo memoryInfo = new MemoryInfo(-1, -1, -1, -1);
+  private MemoryInfo memoryInfo =
+      new MemoryInfo()
+          .setTotalInKb(-1)
+          .setFreeInKb(-1)
+          .setVirtualMemoryTotalInKb(-1)
+          .setVirtualMemoryUsedInKb(-1);
 
   public LinuxMemoryInfoReader() {
     super(new File(MEMINFO));
@@ -46,7 +51,11 @@ public MemoryInfo readMemoryInfo(String content) {
     long virtualMemoryTotalInKb = readMemoryNumber(lines[virtualMemoryTotalInKbIndex]);
     long virtualMemoryUsedInKb = readMemoryNumber(lines[virtualMemoryUsedInKbIndex]);
 
-    return new MemoryInfo(totalInKb, freeInKb, virtualMemoryTotalInKb, virtualMemoryUsedInKb);
+    return new MemoryInfo()
+        .setTotalInKb(totalInKb)
+        .setFreeInKb(freeInKb)
+        .setVirtualMemoryTotalInKb(virtualMemoryTotalInKb)
+        .setVirtualMemoryUsedInKb(virtualMemoryUsedInKb);
   }
 
   private static long readMemoryNumber(String line) {

agent/agent-profiler/agent-diagnostics/src/main/java/com/microsoft/applicationinsights/diagnostics/collection/libos/os/nop/NoOpMemoryInfoReader.java

@@ -17,7 +17,11 @@ public void update() throws OperatingSystemInteractionException {}
 
   @Override
   public MemoryInfo getMemoryInfo() {
-    return new MemoryInfo(-1, -1, -1, -1);
+    return new MemoryInfo()
+        .setTotalInKb(-1)
+        .setFreeInKb(-1)
+        .setVirtualMemoryTotalInKb(-1)
+        .setVirtualMemoryUsedInKb(-1);
   }
 
   @Override

agent/agent-profiler/agent-diagnostics/src/main/java/com/microsoft/applicationinsights/diagnostics/collection/libos/process/ProcessInfo.java

@@ -3,16 +3,8 @@
 
 package com.microsoft.applicationinsights.diagnostics.collection.libos.process;
 
-import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
-import com.fasterxml.jackson.annotation.JsonTypeInfo;
 import java.util.Map;
 
-@JsonTypeInfo(
-    use = JsonTypeInfo.Id.NAME,
-    include = JsonTypeInfo.As.PROPERTY,
-    property = "type",
-    visible = true)
-@JsonIgnoreProperties(ignoreUnknown = true)
 public interface ProcessInfo extends Comparable<ProcessInfo> {
 
   String getName();

agent/agent-tooling/src/test/java/com/microsoft/applicationinsights/agent/internal/profiler/triggers/AlertConfigParserTest.java

@@ -78,20 +78,31 @@ void saneDataIsParsed() {
   @Test
   void requestTriggerIsBuilt() {
     AlertingConfig.RequestTrigger requestTrigger =
-        new AlertingConfig.RequestTrigger(
-            "test",
-            AlertingConfig.RequestTriggerType.LATENCY,
-            new AlertingConfig.RequestFilter(
-                AlertingConfig.RequestFilterType.NAME_REGEX, "/api/users/.*"),
-            new AlertingConfig.RequestAggregation(
-                AlertingConfig.RequestAggregationType.BREACH_RATIO,
-                7000,
-                new AlertingConfig.RequestAggregationConfig(10000, 10)),
-            new AlertingConfig.RequestTriggerThreshold(
-                AlertingConfig.RequestTriggerThresholdType.GREATER_THAN, 0.75f),
-            new AlertingConfig.RequestTriggerThrottling(
-                AlertingConfig.RequestTriggerThrottlingType.FIXED_DURATION_COOLDOWN, 1800),
-            10);
+        new AlertingConfig.RequestTrigger()
+            .setName("test")
+            .setType(AlertingConfig.RequestTriggerType.LATENCY)
+            .setFilter(
+                new AlertingConfig.RequestFilter()
+                    .setType(AlertingConfig.RequestFilterType.NAME_REGEX)
+                    .setValue("/api/users/.*"))
+            .setAggregation(
+                new AlertingConfig.RequestAggregation()
+                    .setType(AlertingConfig.RequestAggregationType.BREACH_RATIO)
+                    .setWindowSizeMillis(7000)
+                    .setConfiguration(
+                        new AlertingConfig.RequestAggregationConfig()
+                            .setThresholdMillis(10000)
+                            .setMinimumSamples(10)))
+            .setThreshold(
+                new AlertingConfig.RequestTriggerThreshold()
+                    .setType(AlertingConfig.RequestTriggerThresholdType.GREATER_THAN)
+                    .setValue(0.75f))
+            .setThrottling(
+                new AlertingConfig.RequestTriggerThrottling()
+                    .setType(AlertingConfig.RequestTriggerThrottlingType.FIXED_DURATION_COOLDOWN)
+                    .setValue(1800))
+            .setProfileDuration(10);
+
     List<AlertingConfig.RequestTrigger> requestTriggers = new ArrayList<>();
     requestTriggers.add(requestTrigger);