Address the review comments

Author: zurex

exporters/memory/include/opentelemetry/exporters/memory/in_memory_span_exporter.h

@@ -81,12 +81,14 @@ class InMemorySpanExporter final : public opentelemetry::sdk::trace::SpanExporte
   }
 
   /**
-   * Timeout is an optional value containing the timeout of the exporter
+   * Attempt to shut down the in-memory span exporter.
+   * @param timeout Timeout is an optional value containing the timeout of the exporter
    * note: passing custom timeout values is not currently supported for this exporter
    * @return Returns the status of the operation
    */
-  bool Shutdown(std::chrono::microseconds /* timeout */) noexcept override
+  bool Shutdown(std::chrono::microseconds timeout) noexcept override
   {
+    (void)timeout;
     is_shutdown_ = true;
     return true;
   }

sdk/include/opentelemetry/sdk/trace/recordable.h

@@ -56,10 +56,10 @@ class Recordable
 
   /**
    * Set an attribute of a span.
-   * @param name the name of the attribute
+   * @param key the name of the attribute
    * @param value the attribute value
    */
-  virtual void SetAttribute(nostd::string_view name,
+  virtual void SetAttribute(nostd::string_view key,
                             const opentelemetry::common::AttributeValue &value) noexcept = 0;
 
   /**