commit 386aa98d9cf1ae52436ce0cdb026fa37163e8337
author Bill Graham <billg@twitter.com> Fri Mar 31 08:13:57 2017 -0700
committer Huijun Wu <huijunw@twitter.com> Mon Apr 17 15:47:15 2017 -0700
tree 0cefe5f71ba74a7c65c779683d6969ede4ed3bd2
parent 8eecf12f13a65b567f91ce70b6edd37073b31fbb

Revert removal of setters that take long bytes (#1786)

* Revert removal of setters that take long bytes

* fix bad html in comments

heron/api/src/java/com/twitter/heron/api/Config.java

diff --git a/heron/api/src/java/com/twitter/heron/api/Config.java b/heron/api/src/java/com/twitter/heron/api/Config.java
index 412c1c6..23af1ee 100644
--- a/heron/api/src/java/com/twitter/heron/api/Config.java
+++ b/heron/api/src/java/com/twitter/heron/api/Config.java
@@ -279,10 +279,30 @@
     conf.put(Config.TOPOLOGY_CONTAINER_CPU_REQUESTED, Float.toString(ncpus));
   }
 
+  /**
+   * Users should use the version of this method at uses ByteAmount
+   * @deprecated use
+   * setContainerDiskRequested(Map<String, Object> conf, ByteAmount nbytes)
+   */
+  @Deprecated
+  public static void setContainerDiskRequested(Map<String, Object> conf, long nbytes) {
+    setContainerDiskRequested(conf, ByteAmount.fromBytes(nbytes));
+  }
+
   public static void setContainerDiskRequested(Map<String, Object> conf, ByteAmount nbytes) {
     conf.put(Config.TOPOLOGY_CONTAINER_DISK_REQUESTED, Long.toString(nbytes.asBytes()));
   }
 
+  /**
+   * Users should use the version of this method at uses ByteAmount
+   * @deprecated use
+   * setContainerRamRequested(Map&lt;String, Object&gt; conf, ByteAmount nbytes)
+   */
+  @Deprecated
+  public static void setContainerRamRequested(Map<String, Object> conf, long nbytes) {
+    setContainerRamRequested(conf, ByteAmount.fromBytes(nbytes));
+  }
+
   public static void setContainerRamRequested(Map<String, Object> conf, ByteAmount nbytes) {
     conf.put(Config.TOPOLOGY_CONTAINER_RAM_REQUESTED, Long.toString(nbytes.asBytes()));
   }
@@ -315,6 +335,17 @@
     return TypeUtils.getListOfStrings(conf.get(Config.TOPOLOGY_AUTO_TASK_HOOKS));
   }
 
+  /**
+   * Users should use the version of this method at uses ByteAmount
+   * @deprecated use
+   * setComponentRam(Map&lt;String, Object&gt; conf, String component, ByteAmount ramInBytes)
+   */
+  @Deprecated
+  public static void setComponentRam(Map<String, Object> conf,
+                                     String component, long ramInBytes) {
+    setComponentRam(conf, component, ByteAmount.fromBytes(ramInBytes));
+  }
+
   public static void setComponentRam(Map<String, Object> conf,
                                      String component, ByteAmount ramInBytes) {
     if (conf.containsKey(Config.TOPOLOGY_COMPONENT_RAMMAP)) {