Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix order of experimental option
Browse files Browse the repository at this point in the history
karthikeyann committed Sep 25, 2024

Verified

This commit was signed with the committer’s verified signature.
karthikeyann Karthikeyan
1 parent c02193d commit 4dbbaa5
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions java/src/main/java/ai/rapids/cudf/JSONOptions.java
Original file line number Diff line number Diff line change
@@ -39,8 +39,8 @@ public final class JSONOptions extends ColumnFilterOptions {
private final boolean allowNonNumericNumbers;
private final boolean allowUnquotedControlChars;
private final boolean cudfPruneSchema;
private final byte lineDelimiter;
private final boolean experimental;
private final byte lineDelimiter;

private JSONOptions(Builder builder) {
super(builder);
@@ -56,8 +56,8 @@ private JSONOptions(Builder builder) {
allowNonNumericNumbers = builder.allowNonNumericNumbers;
allowUnquotedControlChars = builder.allowUnquotedControlChars;
cudfPruneSchema = builder.cudfPruneSchema;
lineDelimiter = builder.lineDelimiter;
experimental = builder.experimental;
lineDelimiter = builder.lineDelimiter;
}

public boolean shouldCudfPruneSchema() {
@@ -140,9 +140,9 @@ public static final class Builder extends ColumnFilterOptions.Builder<JSONOptio

private boolean mixedTypesAsStrings = false;
private boolean keepQuotes = false;
private boolean experimental = false;

private boolean cudfPruneSchema = false;
private boolean experimental = false;
private byte lineDelimiter = '\n';

public Builder withCudfPruneSchema(boolean prune) {
2 changes: 1 addition & 1 deletion java/src/main/java/ai/rapids/cudf/Table.java
Original file line number Diff line number Diff line change
@@ -1445,8 +1445,8 @@ public static TableWithMeta readAndInferJSON(JSONOptions opts, DataSource ds) {
opts.leadingZerosAllowed(),
opts.nonNumericNumbersAllowed(),
opts.unquotedControlChars(),
opts.getLineDelimiter(),
opts.experimental(),
opts.getLineDelimiter(),
dsHandle));
return twm;
} finally {

0 comments on commit 4dbbaa5

Please sign in to comment.