From 130c716105017c7b20b4779973b915968b3dc322 Mon Sep 17 00:00:00 2001
From: Jan Srnicek <jsrnicek@cisco.com>
Date: Wed, 16 Aug 2017 09:21:24 +0200
Subject: HONEYCOMB-386 - Make update optional

If customizer does not support update directly,
updates for its handled nodes are broken up to delete + create pairs.

Change-Id: I2929109e8c9a1db0bef108367cf7d839135ce173
Signed-off-by: Jan Srnicek <jsrnicek@cisco.com>
---
 .../translate/impl/write/GenericWriter.java        | 24 +++++++++++++++++++++-
 1 file changed, 23 insertions(+), 1 deletion(-)

(limited to 'infra/translate-impl/src/main/java/io/fd/honeycomb/translate/impl/write/GenericWriter.java')

diff --git a/infra/translate-impl/src/main/java/io/fd/honeycomb/translate/impl/write/GenericWriter.java b/infra/translate-impl/src/main/java/io/fd/honeycomb/translate/impl/write/GenericWriter.java
index 30d15e370..086936e38 100644
--- a/infra/translate-impl/src/main/java/io/fd/honeycomb/translate/impl/write/GenericWriter.java
+++ b/infra/translate-impl/src/main/java/io/fd/honeycomb/translate/impl/write/GenericWriter.java
@@ -23,18 +23,40 @@ import io.fd.honeycomb.translate.write.WriteFailedException;
 import javax.annotation.Nonnull;
 import org.opendaylight.yangtools.yang.binding.DataObject;
 import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 /**
  * Generic writer with customizable behavior thanks to injected customizer.
  */
 public final class GenericWriter<D extends DataObject> extends AbstractGenericWriter<D> {
 
+    private static final Logger LOG = LoggerFactory.getLogger(GenericWriter.class);
+    private static final String UPDATE_M = "updateCurrentAttributes";
     private final WriterCustomizer<D> customizer;
 
     public GenericWriter(@Nonnull final InstanceIdentifier<D> type,
                          @Nonnull final WriterCustomizer<D> customizer) {
-        super(type);
+        super(type, isUpdateSupported(customizer));
         this.customizer = customizer;
+
+    }
+
+    static boolean isUpdateSupported(final @Nonnull WriterCustomizer<?> customizer) {
+        try {
+            // if customizer overrides updateCurrentAttributes method, it will be used, otherwise updates will be broken into individual
+            // delete + create pairs
+            final Class<? extends WriterCustomizer> customizerClass = customizer.getClass();
+            final Class<?> updateDeclaringClass = customizerClass
+                    .getMethod(UPDATE_M, InstanceIdentifier.class, DataObject.class, DataObject.class, WriteContext.class)
+                    .getDeclaringClass();
+            final boolean supportsUpdate = !WriterCustomizer.class.equals(updateDeclaringClass);
+            LOG.debug("Customizer {} update support : {}|Update declaring class {}", customizerClass, supportsUpdate,
+                    updateDeclaringClass);
+            return supportsUpdate;
+        } catch (NoSuchMethodException e) {
+            throw new IllegalStateException("Unable to detect if customizer supports update", e);
+        }
     }
 
     @Override
-- 
cgit 1.2.3-korg