addressed review comments

Author: SadiHassan

src/Parsers/Kusto/KustoFunctions/KQLStringFunctions.cpp

@@ -376,17 +376,7 @@ bool MakeString::convertImpl(String & out, IParser::Pos & pos)
 
 bool NewGuid::convertImpl(String & out, IParser::Pos & pos)
 {
-    const String fn_name = getKQLFunctionName(pos);
-    if (fn_name.empty())
-        return false;
-    ++pos;
-    ++pos;
-    if (!pos->isEnd() && pos->type != TokenType::Semicolon && pos->type != TokenType::PipeMark)
-        throw Exception(ErrorCodes::BAD_ARGUMENTS, "Function {} expects 0 argument(s)", fn_name);
-    --pos;
-    --pos;
-    --pos;
-    return directMapping(out, pos, "generateUUIDv4");
+    return directMapping(out, pos, "generateUUIDv4", {0, 0});
 }
 
 bool ParseCSV::convertImpl(String & out, IParser::Pos & pos)

tests/queries/0_stateless/02366_kql_func_string.reference

@@ -506,3 +506,4 @@ $-অ-P-WXY
 PQRSTUVWXYZ
 PQ
 {م爱🐂অ
+-- new_guid --

tests/queries/0_stateless/02366_kql_func_string.sql

@@ -490,5 +490,5 @@ print str = make_string(range(80, 85), 86, 87, range(88, 90));
 print str = make_string(dynamic([]), 80, 81);
 print str = make_string(123, 1605, 29233 ,128002, 2437);
 
--- print t = new_guid(1)
--- Code: 36. DB::Exception: Function new_guid expects 0 argument(s). (BAD_ARGUMENTS)
+print '-- new_guid --';
+print t = new_guid(1) -- { clientError NUMBER_OF_ARGUMENTS_DOESNT_MATCH }