Back out "Switch SerialExecutor to default to the immutable global executor"

Summary:
Original commit changeset: b48fe4683165

Our application can't terminate since this change was committed. 100% reproducible.

Reviewed By: WillerZ

Differential Revision: D26621716

fbshipit-source-id: 571ecd9a6f5286469877bf510f1f55b036df9217

folly/executors/SerialExecutor.h

@@ -58,7 +58,7 @@ class SerialExecutor : public SequencedExecutor {
   SerialExecutor& operator=(SerialExecutor&&) = delete;
 
   static KeepAlive<SerialExecutor> create(
-      KeepAlive<Executor> parent = getGlobalCPUExecutor());
+      KeepAlive<Executor> parent = getKeepAliveToken(getCPUExecutor().get()));
 
   class Deleter {
    public:
@@ -76,7 +76,7 @@ class SerialExecutor : public SequencedExecutor {
 
   using UniquePtr = std::unique_ptr<SerialExecutor, Deleter>;
   [[deprecated("Replaced by create")]] static UniquePtr createUnique(
-      std::shared_ptr<Executor> parent);
+      std::shared_ptr<Executor> parent = getCPUExecutor());
 
   /**
    * Add one task for execution in the parent executor