trafodion icon indicating copy to clipboard operation
trafodion copied to clipboard

Published 20 hours ago • verified publisher icon apache

Trafodion 3084 support to create comment on hbase table

Open gdgy opened this issue 6 years ago • 22 comments

gdgy avatar Jun 09 '18 12:06 gdgy

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2734/

Traf-Jenkins avatar Jun 09 '18 12:06 Traf-Jenkins

Test Passed. https://jenkins.esgyn.com/job/Check-PR-master/2734/

Traf-Jenkins avatar Jun 09 '18 15:06 Traf-Jenkins

@anoopsharma00 Does this look ok ? Looks ok to me and can merge with your +1.

sandhyasun avatar Jun 11 '18 22:06 sandhyasun

Were testcases added to regressions to test this feature? Would this work for hbase "ROW", "CELL", "MAP" formats? They all have HBASE as catalog name. Would 'drop hbase table' command remove added comments? Would unregister command remove comments? It will remove the added objectuid for that hbase table. It will also be better to use 'if not registered' instead of 'if not exists' construct. They both are supported.

anoopsharma00 avatar Jun 13 '18 18:06 anoopsharma00

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2761/

Traf-Jenkins avatar Jun 19 '18 02:06 Traf-Jenkins

Test Failed. https://jenkins.esgyn.com/job/Check-PR-master/2761/

Traf-Jenkins avatar Jun 19 '18 02:06 Traf-Jenkins

Previous Test Aborted. New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2762/

Traf-Jenkins avatar Jun 19 '18 02:06 Traf-Jenkins

Test Passed. https://jenkins.esgyn.com/job/Check-PR-master/2762/

Traf-Jenkins avatar Jun 19 '18 05:06 Traf-Jenkins

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2769/

Traf-Jenkins avatar Jun 20 '18 10:06 Traf-Jenkins

Test Passed. https://jenkins.esgyn.com/job/Check-PR-master/2769/

Traf-Jenkins avatar Jun 20 '18 13:06 Traf-Jenkins

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2773/

Traf-Jenkins avatar Jun 21 '18 08:06 Traf-Jenkins

Test Passed. https://jenkins.esgyn.com/job/Check-PR-master/2773/

Traf-Jenkins avatar Jun 21 '18 10:06 Traf-Jenkins

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2779/

Traf-Jenkins avatar Jun 25 '18 03:06 Traf-Jenkins

Test Passed. https://jenkins.esgyn.com/job/Check-PR-master/2779/

Traf-Jenkins avatar Jun 25 '18 06:06 Traf-Jenkins

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2880/

Traf-Jenkins avatar Jul 27 '18 07:07 Traf-Jenkins

Test Failed. https://jenkins.esgyn.com/job/Check-PR-master/2880/

Traf-Jenkins avatar Jul 27 '18 16:07 Traf-Jenkins

Can one of the admins verify this patch?

Traf-Jenkins avatar Aug 01 '18 20:08 Traf-Jenkins

jenkins, retest

moscowgentalman avatar Aug 19 '18 10:08 moscowgentalman

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2951/

Traf-Jenkins avatar Aug 19 '18 10:08 Traf-Jenkins

Test Passed. https://jenkins.esgyn.com/job/Check-PR-master/2951/

Traf-Jenkins avatar Aug 19 '18 13:08 Traf-Jenkins

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/3203/

Traf-Jenkins avatar May 08 '19 14:05 Traf-Jenkins

Test Passed. https://jenkins.esgyn.com/job/Check-PR-master/3203/

Traf-Jenkins avatar May 08 '19 16:05 Traf-Jenkins