Skip to content

Commit

Permalink
Add legacy constructor to CsvParseSpec and DelimitedParseSpec for bac…
Browse files Browse the repository at this point in the history
…kward compatibility (apache#4388)

* Add legacy constructor to CsvParseSpec

* Remove JsonProperty annotations

* Add legacy constructor to DelimitedParseSpec
  • Loading branch information
jihoonson authored and jon-wei committed Jun 9, 2017
1 parent 2cd91b6 commit bbc307b
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 0 deletions.
11 changes: 11 additions & 0 deletions api/src/main/java/io/druid/data/input/impl/CSVParseSpec.java
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,17 @@ public CSVParseSpec(
}
}

@Deprecated
public CSVParseSpec(
TimestampSpec timestampSpec,
DimensionsSpec dimensionsSpec,
String listDelimiter,
List<String> columns
)
{
this(timestampSpec, dimensionsSpec, listDelimiter, columns, false, 0);
}

@JsonProperty
public String getListDelimiter()
{
Expand Down
12 changes: 12 additions & 0 deletions api/src/main/java/io/druid/data/input/impl/DelimitedParseSpec.java
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,18 @@ public DelimitedParseSpec(
}
}

@Deprecated
public DelimitedParseSpec(
TimestampSpec timestampSpec,
DimensionsSpec dimensionsSpec,
String delimiter,
String listDelimiter,
List<String> columns
)
{
this(timestampSpec, dimensionsSpec, delimiter, listDelimiter, columns, false, 0);
}

@JsonProperty("delimiter")
public String getDelimiter()
{
Expand Down

0 comments on commit bbc307b

Please sign in to comment.