[ 
https://issues.apache.org/jira/browse/PHOENIX-7022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17760058#comment-17760058
 ] 

ASF GitHub Bot commented on PHOENIX-7022:
-----------------------------------------

lokiore commented on code in PR #1660:
URL: https://github.com/apache/phoenix/pull/1660#discussion_r1309146926


##########
phoenix-core/src/main/java/org/apache/phoenix/schema/MetaDataClient.java:
##########
@@ -3661,7 +3681,7 @@ private MutationCode processMutationResult(String 
schemaName, String tableName,
             
.setSchemaName(schemaName).setTableName(tableName).setFamilyName(familyName).setColumnName(columnName).setMessage(msg).build().buildException();
         case UNALLOWED_SCHEMA_MUTATION:
             throw new SQLExceptionInfo.Builder(
-                    SQLExceptionCode.CANNOT_SET_OR_ALTER_PHOENIX_TTL)
+                    SQLExceptionCode.CANNOT_SET_OR_ALTER_TTL)

Review Comment:
   We are using that Mutation Code only in one place when we validate 
PhoenixTTLAttribiteSettingForView from our previous View TTL design :- 
https://github.com/apache/phoenix/blob/c4b1af2cb4ceacfb5453285c4b61c49e6954cdb6/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/MetaDataEndpointImpl.java#L3123
   I agree this can be renamed to something related to TTL.





> Add new columns TTL and ROW_KEY_PREFIX
> --------------------------------------
>
>                 Key: PHOENIX-7022
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-7022
>             Project: Phoenix
>          Issue Type: Sub-task
>            Reporter: Jacob Isaac
>            Assignee: Lokesh Khurana
>            Priority: Major
>
> When a view statement is defined by the constraints articulated in 
> PHOENIX-4555, all rows created by the view will be prefixed by a KeyRange. 
> The view thus can simply be represented by the prefixed KeyRange generated by 
> the expression representing the view statement. In other words, there exists 
> a one-to-one mapping between the view (defined by tenant, schema, tablename) 
> and PREFIXED KeyRange.
> For lookup on the PREFIXED KeyRange we will create a new column 
> ROW_KEY_PREFIX in SYSTEM.CATALOG. This new column will be populated during 
> view creation when TTL is specified.
>  
> The TTL column (INTEGER) will store the TTL when specified in line with the 
> HBase spec (which uses an int). The PHOENIX_TTL-related columns and code will 
> be deprecated in a separate jira.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to