SQL Indexes names should be prefixed by schema

[Sébastien Ros]

They are already prefixed with the TablePrefix, they should also use the schema name since they are defined across schemas.

yessql/src/YesSql.Core/Sql/Schema/AlterTableCommand.cs

Lines 55 to 75 in c0771eb

 public void CreateIndex(string indexName, params string[] columnNames) 
 { 
     if (_dialect.PrefixIndex) 
     { 
         indexName = _tablePrefix + indexName; 
     } 
  
     var command = new AddIndexCommand(Name, _dialect.FormatIndexName(indexName), columnNames); 
     TableCommands.Add(command); 
 } 
  
 public void DropIndex(string indexName) 
 { 
     if (_dialect.PrefixIndex) 
     { 
         indexName = _tablePrefix + indexName; 
     } 
  
     var command = new DropIndexCommand(Name, _dialect.FormatIndexName(indexName)); 
     TableCommands.Add(command); 
 } 

[Hisham Bin Ateya]

Shall we use dot notation between schema & table prefix?

[Hisham Bin Ateya]

Is it resolved?