yaooqinn commented on a change in pull request #29635:
URL: https://github.com/apache/spark/pull/29635#discussion_r508950659
##########
File path: docs/sql-migration-guide.md
##########
@@ -40,6 +40,8 @@ license: |
- 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`.
+ - In Spark 3.1, incomplete interval literals, e.g. `INTERVAL '1'`, `INTERVAL
'1 DAY 2'` will fail with IllegalArgumentException. In Spark 3.0, they result
`NULL`s.
Review comment:
OK, I will raise a PR today~
----------------------------------------------------------------
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]
