[ 
https://issues.apache.org/jira/browse/PHOENIX-5654?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Nitesh Maheshwari updated PHOENIX-5654:
---------------------------------------
    Description: 
While working on PHOENIX-5634, I noticed that the connection level property 
'phoenix.default.update.cache.frequency' is being read in 
'MetadataClient::createTable()' and 'Metadata::createTableInternal()' as 
follows:
{code:java}
long updateCacheFrequency = connection.getQueryServices().getProps().getLong(
    QueryServices.DEFAULT_UPDATE_CACHE_FREQUENCY_ATRRIB, 
QueryServicesOptions.DEFAULT_UPDATE_CACHE_FREQUENCY);
{code}
However, looking at the documentation for option 'UPDATE_CACHE_FREQUENCY' at 
[https://phoenix.apache.org/language/index.html], the value for this config 
could also be set to strings 'ALWAYS' and 'NEVER'. The use of getLong() above 
will always return 'QueryServicesOptions.DEFAULT_UPDATE_CACHE_FREQUENCY' when 
the config is set to 'ALWAYS'/'NEVER'. Reading the connection-level property 
should also follow the way the table-level property is read in 
'TableProperty.UPDATE_CACHE_FREQUENCY'.

  was:
While working on PHOENIX-5634, I noticed that the connection level property 
`phoenix.default.update.cache.frequency` is being read in 
`MetadataClient::createTable()` and `Metadata::createTableInternal()` as 
follows:
{code:java}
long updateCacheFrequency = connection.getQueryServices().getProps().getLong(
    QueryServices.DEFAULT_UPDATE_CACHE_FREQUENCY_ATRRIB, 
QueryServicesOptions.DEFAULT_UPDATE_CACHE_FREQUENCY);
{code}
However, looking at the documentation for option `UPDATE_CACHE_FREQUENCY` at 
[https://phoenix.apache.org/language/index.html], the value for this config 
could also be set to strings `ALWAYS` and `NEVER`. The use of getLong() above 
will always return `QueryServicesOptions.DEFAULT_UPDATE_CACHE_FREQUENCY` when 
the config is set to `ALWAYS`/`NEVER`. Reading the connection-level property 
should also follow the way the table-level property is read in 
`TableProperty.UPDATE_CACHE_FREQUENCY`.


> String values (ALWAYS and NEVER) don't work for connection level config 
> phoenix.default.update.cache.frequency
> --------------------------------------------------------------------------------------------------------------
>
>                 Key: PHOENIX-5654
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-5654
>             Project: Phoenix
>          Issue Type: Bug
>    Affects Versions: 4.15.0
>            Reporter: Nitesh Maheshwari
>            Assignee: Nitesh Maheshwari
>            Priority: Major
>             Fix For: 4.15.1
>
>
> While working on PHOENIX-5634, I noticed that the connection level property 
> 'phoenix.default.update.cache.frequency' is being read in 
> 'MetadataClient::createTable()' and 'Metadata::createTableInternal()' as 
> follows:
> {code:java}
> long updateCacheFrequency = connection.getQueryServices().getProps().getLong(
>     QueryServices.DEFAULT_UPDATE_CACHE_FREQUENCY_ATRRIB, 
> QueryServicesOptions.DEFAULT_UPDATE_CACHE_FREQUENCY);
> {code}
> However, looking at the documentation for option 'UPDATE_CACHE_FREQUENCY' at 
> [https://phoenix.apache.org/language/index.html], the value for this config 
> could also be set to strings 'ALWAYS' and 'NEVER'. The use of getLong() above 
> will always return 'QueryServicesOptions.DEFAULT_UPDATE_CACHE_FREQUENCY' when 
> the config is set to 'ALWAYS'/'NEVER'. Reading the connection-level property 
> should also follow the way the table-level property is read in 
> 'TableProperty.UPDATE_CACHE_FREQUENCY'.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to