Uploaded image for project: 'Flink'
  1. Flink
  2. FLINK-4506

CsvOutputFormat defaults allowNullValues to false, even though doc and declaration says true

    Details

      Description

      In the constructor, it has this

      this.allowNullValues = false;
      

      But in the setAllowNullValues() method, the doc says the allowNullValues is true by default. Also, in the declaration of allowNullValues, the value is set to true. It probably makes the most sense to change the constructor.

      	/**
      	 * Configures the format to either allow null values (writing an empty field),
      	 * or to throw an exception when encountering a null field.
      	 * <p>
      	 * by default, null values are allowed.
      	 *
      	 * @param allowNulls Flag to indicate whether the output format should accept null values.
      	 */
      	public void setAllowNullValues(boolean allowNulls) {
      		this.allowNullValues = allowNulls;
      	}
      

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                kirill-morozov-epam Kirill Morozov
                Reporter:
                mwong Michael Wong
              • Votes:
                1 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: