Skip to content

Commit

Permalink
Merge pull request #86 from github/caol-ila-rva-fix
Browse files Browse the repository at this point in the history
Restore RVA compatibility
  • Loading branch information
caol-ila authored Oct 17, 2024
2 parents e872aa4 + dd767f8 commit 7684697
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
4 changes: 3 additions & 1 deletion KustoSchemaTools/Model/Table.cs
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,9 @@ public class Table : IKustoBaseEntity
public string DocString { get; set; }
[Obsolete("Use policies instead")]
public string? RowLevelSecurity { get; set; }

[Obsolete("Use policies instead")]
public bool RestrictedViewAccess { get; set; } = false;

public List<DatabaseScriptContainer> CreateScripts(string name, bool isNew)
{
var scripts = new List<DatabaseScriptContainer>();
Expand Down
3 changes: 3 additions & 0 deletions KustoSchemaTools/Parser/DatabaseCleanup.cs
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,8 @@ public void CleanUp(Database database)
policy.RowLevelSecurity = entity.Value.RowLevelSecurity;
}

policy.RestrictedViewAccess |= entity.Value.RestrictedViewAccess;

if (policy.Retention == database.DefaultRetentionAndCache.Retention)
{
policy.Retention = null;
Expand Down Expand Up @@ -110,6 +112,7 @@ public void CleanUp(Database database)
policy.RowLevelSecurity = entity.Value.RowLevelSecurity;
}


if (policy.Retention == database.DefaultRetentionAndCache.Retention)
{
policy.Retention = null;
Expand Down

0 comments on commit 7684697

Please sign in to comment.