GitHub user BinShi-SecularBird opened a pull request:

    https://github.com/apache/phoenix/pull/351

    PHOENIX-4008: UPDATE STATISTIC should run raw scan with all versions of 
cells.

    @twdsilva @karanmehta93 
    
    The test should fail at the assertion showed below, but it didn't. The 
debugging shows that, even in "UPDATE STATISTICS" command we set the flag to 
read all versions of cells and pass to the scanner, the scanner still only 
returns the latest version of the cell. Could you help to check whether the 
change I made and the test case is correct? I'm correctly debugging the 
internal.
    
                // Because I updated the values of the row with primary key 100,
                // I expected now the estimated bytes should include two 
versions of the cells
                // and the following assertion should fail, but the assertion 
actually succeed.
                assertEquals((Long) 30L, info.getEstimatedBytes());

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

    $ git pull https://github.com/BinShi-SecularBird/phoenix PHOENIX-4008

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

    https://github.com/apache/phoenix/pull/351.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 #351
    
----
commit f4b9f2c2eb6151806c6e655b7c01d5e33a212d14
Author: Bin <bshi@...>
Date:   2018-09-17T19:50:46Z

    PHOENIX-4008: UPDATE STATISTIC should run raw scan with all versions of 
cells.

----


---

Reply via email to