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

Bug in CommonCalc's estimateRowCount() method

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.3.0
    • Component/s: Table API & SQL
    • Labels:
      None

      Description

      (rowCnt * 0.75).min(1.0) should be changed to (rowCnt * 0.75).max(1.0)

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user godfreyhe opened a pull request:

          https://github.com/apache/flink/pull/3740

          FLINK-6327 [table] Bug in CommonCalc's estimateRowCount() method

          (rowCnt * 0.75).min(1.0) should be changed to (rowCnt * 0.75).max(1.0)

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/godfreyhe/flink FLINK-6327

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/flink/pull/3740.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #3740


          commit e6d154d08a353a268d497fa893cac5f21670c639
          Author: godfreyhe <godfreyhe@163.com>
          Date: 2017-04-19T11:43:25Z

          FLINK-6327 [table] Bug in CommonCalc's estimateRowCount() method


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user godfreyhe opened a pull request: https://github.com/apache/flink/pull/3740 FLINK-6327 [table] Bug in CommonCalc's estimateRowCount() method (rowCnt * 0.75).min(1.0) should be changed to (rowCnt * 0.75).max(1.0) You can merge this pull request into a Git repository by running: $ git pull https://github.com/godfreyhe/flink FLINK-6327 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/3740.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #3740 commit e6d154d08a353a268d497fa893cac5f21670c639 Author: godfreyhe <godfreyhe@163.com> Date: 2017-04-19T11:43:25Z FLINK-6327 [table] Bug in CommonCalc's estimateRowCount() method
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user fhueske commented on the issue:

          https://github.com/apache/flink/pull/3740

          +1 to merge

          Show
          githubbot ASF GitHub Bot added a comment - Github user fhueske commented on the issue: https://github.com/apache/flink/pull/3740 +1 to merge
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/flink/pull/3740

          Show
          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/3740

            People

            • Assignee:
              godfreyhe godfrey he
              Reporter:
              godfreyhe godfrey he
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development