Given statefulness we maintain around parquetColumnName, we should no…

…t skip the logic when a user explicitly sets the parquet column name the same as the column name
devinrsmith · Oct 1, 2024 · 6b58468 · 6b58468
1 parent 9fa979e
commit 6b58468
Showing 1 changed file with 0 additions and 3 deletions.
diff --git a/extensions/parquet/table/src/main/java/io/deephaven/parquet/table/ParquetInstructions.java b/extensions/parquet/table/src/main/java/io/deephaven/parquet/table/ParquetInstructions.java
@@ -797,9 +797,6 @@ public Builder(final ParquetInstructions parquetInstructions) {
  }
 
  public Builder addColumnNameMapping(final String parquetColumnName, final String columnName) {
- if (parquetColumnName.equals(columnName)) {
- return this;
- }
  final ColumnInstructions ci = getColumnInstructions(columnName);
  ci.setParquetColumnName(parquetColumnName);
  if (parquetColumnNameToInstructions == null) {