Skip to content

Commit 31e7198

Browse files
committed
use switch case to check version
1 parent a09240c commit 31e7198

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

pkg/ottl/ottlfuncs/func_murmurhash3.go

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,14 @@ func createMurmurHash3Function[K any](_ ottl.FunctionContext, oArgs ottl.Argumen
3737

3838
version := v128
3939
if !args.Version.IsEmpty() {
40-
if (args.Version.Get() != v32) && (args.Version.Get() != v128) {
41-
return nil, fmt.Errorf("invalid arguments: %s. Version should be either \"32\" or \"128\"", args.Version.Get())
40+
v := args.Version.Get()
41+
42+
switch v {
43+
case v32, v128:
44+
version = v
45+
default:
46+
return nil, fmt.Errorf("invalid arguments: %s. Version should be either \"32\" or \"128\"", v)
4247
}
43-
version = args.Version.Get()
4448
}
4549

4650
return MurmurHash3HexString(args.Target, version)

0 commit comments

Comments
 (0)