Fix parameter substitution in Package pragma (#11055)

Author: antoniox

ydb/library/yql/providers/yt/lib/yt_url_lister/yt_url_lister.cpp

@@ -23,26 +23,29 @@ class TYtUrlLister: public IUrlLister {
     TYtUrlLister() = default;
 
 public:
-    bool Accept(const THttpURL& url) const override {
-        auto rawScheme = url.GetField(NUri::TField::FieldScheme);
+    bool Accept(const TString& url) const override {
+        auto httpUrl = ParseURL(url);
+        auto rawScheme = httpUrl.GetField(NUri::TField::FieldScheme);
         return NUri::EqualNoCase(rawScheme, Scheme);
     }
 
-    TVector<TUrlListEntry> ListUrl(const THttpURL& url, const TString& token) const override {
+    TVector<TUrlListEntry> ListUrl(const TString& url, const TString& token) const override {
         InitYtApiOnce();
 
-        TCgiParameters params(url.GetField(NUri::TField::FieldQuery));
+        auto httpUrl = ParseURL(url);
+
+        TCgiParameters params(httpUrl.GetField(NUri::TField::FieldQuery));
 
         NYT::TCreateClientOptions createOpts;
         if (token) {
             createOpts.Token(token);
         }
 
-        auto host = url.PrintS(NUri::TField::FlagHostPort);
+        auto host = httpUrl.PrintS(NUri::TField::FlagHostPort);
 
         auto path = params.Has("path")
             ? params.Get("path")
-            : TString(TStringBuf(url.GetField(NUri::TField::FieldPath)).Skip(1));
+            : TString(TStringBuf(httpUrl.GetField(NUri::TField::FieldPath)).Skip(1));
 
         auto client = NYT::CreateClient(host, createOpts);
         NYT::IClientBasePtr tx = client;
@@ -67,7 +70,7 @@ class TYtUrlLister: public IUrlLister {
                 url.Set(NUri::TField::FieldQuery, TStringBuilder() << "transaction_id=" << txId);
             }
 
-            return url;
+            return url.PrintS();
         };
 
         NYT::TListOptions listOpts;

ydb/library/yql/tests/sql/suites/library/p_package_cluster.json

@@ -0,0 +1 @@
+"plato"

ydb/library/yql/tests/sql/suites/library/package.cfg

@@ -2,3 +2,4 @@ providers yt
 yt_file yt://plato/package/total.sql package_total.sql.txt
 yt_file yt://plato/package/detail/foo.sql package_foo.sql.txt
 yt_file yt://plato/package/detail/bar.sql package_bar.sql.txt
+param $cluster p_package_cluster.json

ydb/library/yql/tests/sql/suites/library/package.sql

@@ -1,6 +1,8 @@
 /* ytfile can not */
 
-pragma package("project.package", "yt://plato/package");
+declare $cluster as String;
+
+pragma package("project.package", "yt://{$cluster}/package");
 
 import pkg.project.package.total symbols $do_total;