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

Sameer Abhyankar commented on KUDU-1406:
----------------------------------------

Looking through the code for master-path-handlers.cc, it seems that 
"master_addresses" is set to comma separated "all_addresses" which is never 
initialized to the values in the for (..) loop. This looks like an easy one and 
I would like to work on it :) Can this be assigned to me?

string master_addresses;
  if (master_->opts().IsDistributed()) {
    vector<string> all_addresses;
    for (const HostPort& hp : master_->opts().master_addresses) {
      master_addresses.append(hp.ToString());
    }
    master_addresses = JoinElements(all_addresses, ",");

> Missing kudu.master_addresses in TBLPROPERTIES via Web UI
> ---------------------------------------------------------
>
>                 Key: KUDU-1406
>                 URL: https://issues.apache.org/jira/browse/KUDU-1406
>             Project: Kudu
>          Issue Type: Bug
>    Affects Versions: 0.7.1
>            Reporter: Sameer Abhyankar
>            Priority: Minor
>
> I noticed that after a table is created using the Java APIs or the Impala 
> shell, the IMPALA Create Table statement shown on the Kudu Web UI 
> (someHost:8051/table?id=someTableId) has an empty kudu.master_addresses field:
> Impala CREATE TABLE statement
> CREATE EXTERNAL TABLE `kudu_table4` (
> `c1` INT,
> `c2` INT,
> `c3` STRING,
> `c4` DOUBLE,
> `c5` FLOAT
> )
> TBLPROPERTIES(
>   'storage_handler' = 'com.cloudera.kudu.hive.KuduStorageHandler',
>   'kudu.table_name' = 'kudu_table4',
>   'kudu.master_addresses' = '',
>   'kudu.key_columns' = 'c1'
> );
> If I do a 'desc formatted kudu_table4' I do see the correct properties so 
> probably something in the web UI code (master-path-handlers.cc?) where the 
> master_addresses?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to