haridsv opened a new pull request, #1969:
URL: https://github.com/apache/phoenix/pull/1969

   This PR includes the following changes:
   * Currently IndexMaintainer assumes that both data and index tables have the 
same number of salt buckets. This change adds a separate field for data table 
and uses it for constructing the data keys. This fixes the joins between 
uncovered index and data tables.
   * There is a typo in the CDC mutation code path that uses data table to 
construct the index row key instead of the index table which was throwing off 
the logic that join the changes to the data rows.
   * Test for time range queries has been rewritten to be simpler and also add 
coverage for the above special DF markers.
   * Some of the existing debug code in the CDC IT test has been refactored to 
be less repetitive.


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@phoenix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to