parametrize table_prefix_separator in hbase connection

[Stefano Zilli]

Currently table_prefix_separator is hardcoded to be an underscore. It should be nice to have it customizable.

Something like:
connection = hbase://localhost:9090/?table_prefix=ceilo&table_prefix_separator=.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/100829

[OpenStack Infra (hudson-openstack)]

Change abandoned by Stefano Zilli (<email address hidden>) on branch: master
Review: https://review.openstack.org/100829

[Lianhao Lu (lianhao-lu)]

Seems like Stefano Zilli abandoned the effort?

[Stefano Zilli (szilli)]

Tbh I've got discouraged when my patch was discarded because it was adding a random separator to an already random generated table name (uuid4) in the unit tests (for me adding random to random is still random) and decided to wait. I'm actually still maintaining the patch and using it in production. Maybe I will propose a new fix this time adding something similar to the new prefix string os.getenv("CEILOMETER_TEST_HBASE_TABLE_PREFIX", "test") in the unit tests.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/228363

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/228363
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=3ae2e054ac7338077f364577bc624f917e0d508a
Submitter: Jenkins
Branch: master

commit 3ae2e054ac7338077f364577bc624f917e0d508a
Author: Stefano Zilli <email address hidden>
Date: Mon Sep 28 09:12:44 2015 +0200

    Parametrize table_prefix_separator in hbase

    table_prefix_separator is hardcoded to be an underscore (happybase
    default). This fix adds the possibility to parametrize this value using
    a parameter in ceilometer database configuration. An example is:
    hbase://localhost:9090/?table_prefix=ceilo&table_prefix_separator=.

    Change-Id: I8c3c2a0b35001991844e679282ffd0ee4ae76abc
    Closes-Bug: #1331374

[Thierry Carrez (ttx)]

This issue was fixed in the openstack/ceilometer 6.0.0.0b1 development milestone.