Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cannot version unversioned table (inconsistent view on the matter) #170

Open
strk opened this issue Mar 21, 2019 · 10 comments
Open

cannot version unversioned table (inconsistent view on the matter) #170

strk opened this issue Mar 21, 2019 · 10 comments

Comments

@strk
Copy link
Contributor

strk commented Mar 21, 2019

See this session:

lol_3_17=# select table_version.ver_version();
1.7.0dev 1.6.0-16-g7fabb6f
lol_3_17=# select table_version.ver_is_table_versioned('bde_ext', 'adjustment_run');
f
lol_3_17=# select table_version.ver_enable_versioning('bde_ext', 'adjustment_run');
ERROR:  Table bde_ext.adjustment_run is already versioned

Basically, ver_is_table_versioned says a table is NOT versioned while ver_enable_versioning says the table is ALREADY versioned... annoying enough

@strk strk added this to the 1.7.0 milestone Mar 21, 2019
@strk strk self-assigned this Mar 21, 2019
@strk
Copy link
Contributor Author

strk commented Mar 21, 2019

There's no trigger on the table:

lol_3_17=# \d bde_ext.adjustment_run
                Table "bde_ext.adjustment_run"
       Column       |            Type             | Modifiers 
--------------------+-----------------------------+-----------
 id                 | integer                     | not null
 adm_id             | integer                     | not null
 cos_id             | integer                     | not null
 status             | character varying(4)        | not null
 usr_id_exec        | character varying(20)       | not null
 adjust_datetime    | timestamp without time zone | 
 description        | character varying(100)      | 
 sum_sqrd_residuals | numeric(22,12)              | 
 redundancy         | numeric(22,12)              | 
 wrk_id             | integer                     | 
 audit_id           | integer                     | not null
Indexes:
    "pkey_adjustment_run" PRIMARY KEY, btree (id)

The table_version.versioned_table table is empty.
The only thing is that a table_version.bde_ext_adjustment_run_revision table exists.

What should we do in these cases ?
Shall we consider the table as versioned or non-versioned ?

@imincik
Copy link
Contributor

imincik commented Mar 21, 2019

I think it should be considered as non-versioned. How has that happen ?

@strk
Copy link
Contributor Author

strk commented Mar 22, 2019 via email

@stale
Copy link

stale bot commented Apr 5, 2019

This issue has been automatically marked as stale as there has not been any activity for sometime. The issue will be closed in 14 days if no further activity.

@stale stale bot added the Stale label Apr 5, 2019
@imincik
Copy link
Contributor

imincik commented Apr 6, 2019

This task is still relevant.

@stale stale bot removed the Stale label Apr 6, 2019
@stale
Copy link

stale bot commented Apr 20, 2019

This issue has been automatically marked as stale as there has not been any activity for sometime. The issue will be closed in 14 days if no further activity.

@stale stale bot added the Stale label Apr 20, 2019
@imincik
Copy link
Contributor

imincik commented Apr 23, 2019

This task is still relevant.

@stale stale bot removed the Stale label Apr 23, 2019
@stale
Copy link

stale bot commented May 7, 2019

This issue has been automatically marked as stale as there has not been any activity for sometime. The issue will be closed in 14 days if no further activity.

@stale stale bot added the Stale label May 7, 2019
@imincik
Copy link
Contributor

imincik commented May 7, 2019

This task is still relevant.

@stale stale bot removed the Stale label May 7, 2019
@strk strk modified the milestones: 1.7.0, 1.8.0 Jul 25, 2019
@strk strk removed this from the 1.8.0 milestone Feb 11, 2020
@strk strk removed their assignment Nov 10, 2020
@billgeo billgeo added wontfix and removed wontfix labels Jun 8, 2022
@billgeo
Copy link
Contributor

billgeo commented Jun 8, 2022

This seems like quite an edge case. Would require someone to drop cascade and then rebuild with schema loader.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants