Skip to content

Commit

Permalink
Remove unused database comment in file metastore
Browse files Browse the repository at this point in the history
It was never set.
  • Loading branch information
findepi committed Sep 19, 2022
1 parent 18d875f commit 23c0eaf
Showing 1 changed file with 0 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -29,21 +29,18 @@ public class DatabaseMetadata
private final Optional<String> writerVersion;
private final Optional<String> ownerName;
private final Optional<PrincipalType> ownerType;
private final Optional<String> comment;
private final Map<String, String> parameters;

@JsonCreator
public DatabaseMetadata(
@JsonProperty("writerVersion") Optional<String> writerVersion,
@JsonProperty("ownerName") Optional<String> ownerName,
@JsonProperty("ownerType") Optional<PrincipalType> ownerType,
@JsonProperty("comment") Optional<String> comment,
@JsonProperty("parameters") Map<String, String> parameters)
{
this.writerVersion = requireNonNull(writerVersion, "writerVersion is null");
this.ownerName = requireNonNull(ownerName, "ownerName is null");
this.ownerType = requireNonNull(ownerType, "ownerType is null");
this.comment = requireNonNull(comment, "comment is null");
this.parameters = ImmutableMap.copyOf(requireNonNull(parameters, "parameters is null"));
}

Expand All @@ -52,7 +49,6 @@ public DatabaseMetadata(String currentVersion, Database database)
this.writerVersion = Optional.of(requireNonNull(currentVersion, "currentVersion is null"));
this.ownerName = database.getOwnerName();
this.ownerType = database.getOwnerType();
this.comment = database.getComment();
this.parameters = database.getParameters();
}

Expand All @@ -74,12 +70,6 @@ public Optional<PrincipalType> getOwnerType()
return ownerType;
}

@JsonProperty
public Optional<String> getComment()
{
return comment;
}

@JsonProperty
public Map<String, String> getParameters()
{
Expand Down

0 comments on commit 23c0eaf

Please sign in to comment.