fix minor unreachable code caused by log.Fatal (#22618)

* fix minor unreachable code caused by log.Fatal

Signed-off-by: Abirdcfly <[email protected]>

* Apply suggestions from code review

Co-authored-by: Danny McCormick <[email protected]>

* Update sdks/python/container/boot.go

Co-authored-by: Danny McCormick <[email protected]>

Signed-off-by: Abirdcfly <[email protected]>
Co-authored-by: Danny McCormick <[email protected]>

Author: Abirdcfly

sdks/go/pkg/beam/core/runtime/harness/worker_status_test.go

@@ -18,12 +18,13 @@ package harness
 import (
 	"context"
 	"fmt"
-	"github.com/apache/beam/sdks/v2/go/pkg/beam/core/runtime/harness/statecache"
 	"log"
 	"net"
 	"strings"
 	"testing"
 
+	"github.com/apache/beam/sdks/v2/go/pkg/beam/core/runtime/harness/statecache"
+
 	fnpb "github.com/apache/beam/sdks/v2/go/pkg/beam/model/fnexecution_v1"
 	"google.golang.org/grpc"
 	"google.golang.org/grpc/credentials/insecure"
@@ -55,7 +56,6 @@ func setup(t *testing.T, srv *BeamFnWorkerStatusServicer) {
 	go func() {
 		if err := server.Serve(lis); err != nil {
 			log.Fatalf("failed to serve: %v", err)
-			panic(err)
 		}
 	}()
 	t.Cleanup(func() {

sdks/python/container/boot.go

@@ -212,7 +212,6 @@ func main() {
 		go func(workerId string) {
 			log.Printf("Executing: python %v", strings.Join(args, " "))
 			log.Fatalf("Python exited: %v", execx.ExecuteEnv(map[string]string{"WORKER_ID": workerId}, "python", args...))
-			wg.Done()
 		}(workerId)
 	}
 	wg.Wait()

sdks/typescript/boot.go

@@ -170,5 +170,4 @@ func main() {
 	}
 
 	log.Fatalf("User program exited: %v", execx.Execute("npx", args...))
-	log.Printf("SDK exited cleanly.")
 }