gjacoby126 commented on a change in pull request #1216:
URL: https://github.com/apache/phoenix/pull/1216#discussion_r626164437



##########
File path: 
phoenix-core/src/it/java/org/apache/phoenix/end2end/index/MutableIndexIT.java
##########
@@ -999,6 +1001,25 @@ public void testDeleteCount_index() throws Exception {
         }
     }
 
+    @Test
+    public void testCreateIndexSchemaVersion() throws Exception {

Review comment:
       Phoenix won't really "use" a schema version at all during a select. To 
be precise, when it grabs the PTable to be able to interpret the query results, 
the schema version will be in the PTable, but it's not really used by the query 
logic. 
   
   The point of schema version is to allow:
   1. Applications to check to see if a table, view or index is compatible with 
a particular version of their schema 
   2. In the future, when we have change streaming from Phoenix, messages will 
be annotated with the schema version so consumers of the message will be able 
to check to see if they can interpret the message (i.e, it's compatible with a 
particular version of a consuming application) or not. 




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to