cloud-fan commented on a change in pull request #29543:
URL: https://github.com/apache/spark/pull/29543#discussion_r477042551



##########
File path: docs/sql-migration-guide.md
##########
@@ -38,7 +38,7 @@ license: |
 
   - In Spark 3.1, when `spark.sql.ansi.enabled` is false, Spark always returns 
null if the sum of decimal type column overflows. In Spark 3.0 or earlier, in 
the case, the sum of decimal type column may return null or incorrect result, 
or even fails at runtime (depending on the actual query plan execution).
 
-  - In Spark 3.1, when loading a dataframe, `path` or `paths` option cannot 
coexist with `load()`'s path parameters. For example, 
`spark.read.format("csv").option("path", "/tmp").load("/tmp2")` or 
`spark.read.option("path", "/tmp").csv("/tmp2")` will throw 
`org.apache.spark.sql.AnalysisException`. In Spark version 3.0 and below, 
`path` option is overwritten if one path parameter is passed to `load()`, or 
`path` option is added to the overall paths if multiple path parameters are 
passed to `load()`. To restore the behavior before Spark 3.1, you can set 
`spark.sql.legacy.pathOptionBehavior.enabled` to `true`.
+  - In Spark 3.1, `path` option cannot coexist when the following methods are 
called with path parameter(s): `DataFrameReader.load()`, 
`DataFrameWriter.save()`, `DataStreamReader.load()`, or 
`DataStreamWriter.start()`. In addition, `paths` option cannot coexist for 
`DataFrameReader.load()`. For example, `spark.read.format("csv").option("path", 
"/tmp").load("/tmp2")` or `spark.read.option("path", "/tmp").csv("/tmp2")` will 
throw `org.apache.spark.sql.AnalysisException`. In Spark version 3.0 and below, 
`path` option is overwritten if one path parameter is passed to above methods; 
`path` option is added to the overall paths if multiple path parameters are 
passed to `DataFrameReader.load()`. To restore the behavior before Spark 3.1, 
you can set `spark.sql.legacy.pathOptionBehavior.enabled` to `true`.

Review comment:
       `In addition, ...` this doesn't seem to provide additional info.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to