This is an automated email from the ASF dual-hosted git repository.
jincheng pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/flink.git
The following commit(s) were added to refs/heads/master by this push:
new 7fafb20 [FLINK-11728] [table] Deprecate CalciteConfig temporarily
7fafb20 is described below
commit 7fafb2080f16a78436636395480c44aac95a112e
Author: Timo Walther <[email protected]>
AuthorDate: Fri Feb 22 17:46:49 2019 +0100
[FLINK-11728] [table] Deprecate CalciteConfig temporarily
This closes #7840
---
.../scala/org/apache/flink/table/api/TableConfig.scala | 16 ++++++++++++++++
.../org/apache/flink/table/calcite/CalciteConfig.scala | 16 ++++++++++++++++
2 files changed, 32 insertions(+)
diff --git
a/flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/api/TableConfig.scala
b/flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/api/TableConfig.scala
index 51c9a37..5f0e6df 100644
---
a/flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/api/TableConfig.scala
+++
b/flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/api/TableConfig.scala
@@ -81,13 +81,29 @@ class TableConfig {
/**
* Returns the current configuration of Calcite for Table API and SQL
queries.
+ *
+ * @deprecated This method will be removed temporarily while the API is
uncoupled
+ * from Calcite. An alternative might be provided in the
future. See FLINK-11728
+ * for more information.
*/
+ @Deprecated
+ @deprecated(
+ "This method will be removed temporarily while the API is uncoupled from
Calcite.",
+ "1.8.0")
def getCalciteConfig: CalciteConfig = calciteConfig
/**
* Sets the configuration of Calcite for Table API and SQL queries.
* Changing the configuration has no effect after the first query has been
defined.
+ *
+ * @deprecated This method will be removed temporarily while the API is
uncoupled
+ * from Calcite. An alternative might be provided in the
future. See FLINK-11728
+ * for more information.
*/
+ @Deprecated
+ @deprecated(
+ "This method will be removed temporarily while the API is uncoupled from
Calcite.",
+ "1.8.0")
def setCalciteConfig(calciteConfig: CalciteConfig): Unit = {
this.calciteConfig = calciteConfig
}
diff --git
a/flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/calcite/CalciteConfig.scala
b/flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/calcite/CalciteConfig.scala
index b5d6c06..064a25c 100644
---
a/flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/calcite/CalciteConfig.scala
+++
b/flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/calcite/CalciteConfig.scala
@@ -33,7 +33,15 @@ import scala.collection.JavaConverters._
/**
* Builder for creating a Calcite configuration.
+ *
+ * @deprecated This class will be removed temporarily while the API is
uncoupled
+ * from Calcite. An alternative might be provided in the future.
See FLINK-11728
+ * for more information.
*/
+@Deprecated
+@deprecated(
+ "This method will be removed temporarily while the API is uncoupled from
Calcite.",
+ "1.8.0")
class CalciteConfigBuilder {
/**
@@ -255,7 +263,15 @@ class CalciteConfigBuilder {
/**
* Calcite configuration for defining a custom Calcite configuration for
Table and SQL API.
+ *
+ * @deprecated This class will be removed temporarily while the API is
uncoupled
+ * from Calcite. An alternative might be provided in the future.
See FLINK-11728
+ * for more information.
*/
+@Deprecated
+@deprecated(
+ "This method will be removed temporarily while the API is uncoupled from
Calcite.",
+ "1.8.0")
trait CalciteConfig {
/**