Try to limit heap size by reusing String references

diff --git a/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/DefaultInputFileValueCoder.java b/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/DefaultInputFileValueCoder.java
index d60cbac0088cbf8ab6bd83fa2f17aaf26eb0c7d5..6dcfc9903b1184b5c5d9e80e11d6781bc358f169 100644 (file)
--- a/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/DefaultInputFileValueCoder.java
+++ b/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/DefaultInputFileValueCoder.java
@@ -35,14 +35,14 @@ class DefaultInputFileValueCoder implements ValueCoder {
   public void put(Value value, Object object, CoderContext context) {
     DeprecatedDefaultInputFile f = (DeprecatedDefaultInputFile) object;
     putUTFOrNull(value, f.relativePath());
-    putUTFOrNull(value, f.getFileBaseDir().toString());
+    value.putString(f.getFileBaseDir().toString());
     putUTFOrNull(value, f.deprecatedKey());
-    putUTFOrNull(value, f.sourceDirAbsolutePath());
+    value.putString(f.sourceDirAbsolutePath());
     putUTFOrNull(value, f.pathRelativeToSourceDir());
     putUTFOrNull(value, f.absolutePath());
-    putUTFOrNull(value, f.language());
-    putUTFOrNull(value, f.type().name());
-    putUTFOrNull(value, f.status().name());
+    value.putString(f.language());
+    value.putString(f.type().name());
+    value.putString(f.status().name());
     putUTFOrNull(value, f.hash());
     value.put(f.lines());
     putUTFOrNull(value, f.key());

diff --git a/sonar-batch/src/main/java/org/sonar/batch/scan/measure/MeasureValueCoder.java b/sonar-batch/src/main/java/org/sonar/batch/scan/measure/MeasureValueCoder.java
index c0906b51d6d760475a4c186b2a2f0d419f339c6f..bd8cee930d83b44e5ba6e46f8a2bd9db733a505e 100644 (file)
--- a/sonar-batch/src/main/java/org/sonar/batch/scan/measure/MeasureValueCoder.java
+++ b/sonar-batch/src/main/java/org/sonar/batch/scan/measure/MeasureValueCoder.java
@@ -48,7 +48,7 @@ class MeasureValueCoder implements ValueCoder {
     value.put(m.getValue());
     putUTFOrNull(value, m.getData());
     putUTFOrNull(value, m.getDescription());
-    putUTFOrNull(value, m.getAlertStatus() != null ? m.getAlertStatus().name() : null);
+    value.putString(m.getAlertStatus() != null ? m.getAlertStatus().name() : null);
     putUTFOrNull(value, m.getAlertText());
     value.put(m.getTendency());
     value.putDate(m.getDate());
@@ -65,7 +65,7 @@ class MeasureValueCoder implements ValueCoder {
     Integer personId = m.getPersonId();
     value.put(personId != null ? personId.intValue() : null);
     PersistenceMode persistenceMode = m.getPersistenceMode();
-    putUTFOrNull(value, persistenceMode != null ? persistenceMode.name() : null);
+    value.putString(persistenceMode != null ? persistenceMode.name() : null);
   }
 
   private void putUTFOrNull(Value value, @Nullable String utfOrNull) {