Let's not expose the underlying Method. (#13543)

Summary:

.

Reviewed By: JacobSzwejbka, mergennachin

Differential Revision: D80595270

Author: shoumikhin

extension/module/module.cpp

@@ -215,12 +215,6 @@ runtime::Error Module::load_method(
   return runtime::Error::Ok;
 }
 
-ET_NODISCARD runtime::Result<Method*> Module::method(
-    const std::string& method_name) {
-  ET_CHECK_OK_OR_RETURN_ERROR(load_method(method_name));
-  return methods_[method_name].method.get();
-}
-
 runtime::Result<MethodMeta> Module::method_meta(
     const std::string& method_name) {
   ET_CHECK_OK_OR_RETURN_ERROR(load());

extension/module/module.h

@@ -205,21 +205,6 @@ class Module {
     return methods_.erase(method_name);
   }
 
-  /**
-   * DEPRECATED: Module manages each Method exclusively.
-   *
-   * Get a method by it's name. Not recommended to use this method directly as
-   * an end user. It's exposed to allow for composability of module in apis that
-   * operate on method.
-   *
-   * @param[in] method_name The name of the method to get.
-   *
-   * @returns A Result object containing either a pointer to the requested
-   *          method or an error to indicate failure.
-   */
-  ET_DEPRECATED ET_NODISCARD runtime::Result<Method*> method(
-      const std::string& method_name);
-
   /**
    * Load the 'forward' method from the program and set up memory management if
    * needed. The loaded method is cached to reuse the next time it's executed.