yanxinyi commented on a change in pull request #799:
URL: https://github.com/apache/phoenix/pull/799#discussion_r437607069
##########
File path: phoenix-protocol/src/main/PTable.proto
##########
@@ -106,9 +106,9 @@ message PTable {
optional int32 viewIndexIdType = 39 [default = 5];
optional bool viewModifiedUpdateCacheFrequency = 40;
optional bool viewModifiedUseStatsForParallelization = 41;
- optional int64 viewTTL = 42;
- optional int64 viewTTLHighWaterMark = 43;
- optional bool viewModifiedViewTTL = 44;
+ optional int64 phoenixTTL = 42;
+ optional int64 phoenixTTLHighWaterMark = 43;
+ optional bool viewModifiedPhoenixTTL = 44;
Review comment:
do we still want to keep the naming of viewModifiedPhoenixTTL? do view
or general phoenix TTL matter here?
----------------------------------------------------------------
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]
