Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/708#discussion_r95047914
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/store/mock/MockGroupScanPOP.java
---
@@ -104,44 +102,46 @@ public String toString() {
}
@JsonInclude(Include.NON_NULL)
- public static class MockColumn{
+ public static class MockColumn {
@JsonProperty("type") public MinorType minorType;
public String name;
public DataMode mode;
public Integer width;
public Integer precision;
public Integer scale;
-
+ public String generator;
+ public Integer repeat;
@JsonCreator
- public MockColumn(@JsonProperty("name") String name,
@JsonProperty("type") MinorType minorType, @JsonProperty("mode") DataMode mode,
@JsonProperty("width") Integer width, @JsonProperty("precision") Integer
precision, @JsonProperty("scale") Integer scale) {
--- End diff --
Same here - for overloading constructor.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---