You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/02/14 14:02:42 UTC
[jira] [Commented] (FLINK-5567) Introduce and migrate current table
statistics to FlinkStatistics
[ https://issues.apache.org/jira/browse/FLINK-5567?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15865805#comment-15865805 ]
ASF GitHub Bot commented on FLINK-5567:
---------------------------------------
Github user fhueske commented on a diff in the pull request:
https://github.com/apache/flink/pull/3197#discussion_r101022558
--- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/stats/FlinkStatistic.scala ---
@@ -0,0 +1,87 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.flink.table.plan.stats
+
+import java.lang.Double
+import java.util.{Collections, List}
+
+import org.apache.calcite.rel.{RelCollation, RelDistribution}
+import org.apache.calcite.schema.Statistic
+import org.apache.calcite.util.ImmutableBitSet
+
+/**
+ * The class is responsible for provide statistics for flink table.
+ *
+ * @param tableStats
+ */
+class FlinkStatistic(tableStats: Option[TableStats]) extends Statistic {
+
+ /**
+ * Get table stats
+ *
+ * @return table stats
+ */
+ def getTableStats: TableStats = tableStats.getOrElse(null)
+
+ /**
+ * Get stats of specified column
+ *
+ * @param columnName which column to get stats
+ * @return stats of specified column
+ */
+ def getColumnStats(columnName: String) = tableStats match {
+ case Some(tStats) => tStats.colStats.getOrElse(columnName, null)
+ case None => null
+ }
+
+ /**
+ * Get number of rows in the table
+ *
+ * @return number of rows in the table
+ */
+ override def getRowCount: Double = tableStats match {
+ case Some(tStats) => tStats.rowCount.toDouble
+ case None => null
+ }
+
+ override def getCollations: List[RelCollation] = Collections.emptyList()
+
+ override def isKey(columns: ImmutableBitSet): Boolean = false
+
+ override def getDistribution: RelDistribution = null
+
+}
+
+/**
+ * Utility functions regarding FlinkStatistic
+ */
+object FlinkStatistic {
--- End diff --
Rename to `FlinkStatistics`
> Introduce and migrate current table statistics to FlinkStatistics
> -----------------------------------------------------------------
>
> Key: FLINK-5567
> URL: https://issues.apache.org/jira/browse/FLINK-5567
> Project: Flink
> Issue Type: Sub-task
> Components: Table API & SQL
> Reporter: Kurt Young
> Assignee: zhangjing
>
> 1. Introduce FlinkStatistic class, which is subclass of Calcite Statistic.
> 2. Integrate FlinkStatistic with FlinkTable
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)