trafodion icon indicating copy to clipboard operation
trafodion copied to clipboard

[TRAFODION-2953] Do not merge

Open moscowgentalman opened this issue 7 years ago • 19 comments

This is the very first code drop for TRAFODION-2953, the major purpose is to get feedback to see if the design is OK and shall we continue with this effort or not. Please check https://issues.apache.org/jira/browse/TRAFODION-2953, I attached a design document there.

This PR contains initial prototype of the design. It still has a major code-refactor to do, but I would like to hear from community first, before I spend more time on it, in case there are something fundamentally wrong.

The major code-refactor I mentioned above is for this prototype code is to add a new PartitionFunction, instead of modifying the SinglePartitionPartitioningFunction directly. I will inherit a new SinglePartitionWithSmallTablePartitioningFunction from SinglePartitionPartitioningFunction next.

One can check the TEST035 to get a demo of how it works.

There are many other tasks following this initial code drop. For example, index/alter/create-super-table etc.

The major idea to support this is to add a new SYSTEM column TBLNM, so in a single Region, rows belong to different tables can be distinguished. When doing IUDS, always add predicate TBLNM = 'theNameOfTheTable' to filter out the correct rows.

Please help to review and let me know your comments, many thanks.

The requirement is strong. In some use cases, there must be many small tables.

moscowgentalman avatar Apr 25 '18 03:04 moscowgentalman

something wrong with my PR, most changes not in the PR. Close it.

moscowgentalman avatar Apr 25 '18 03:04 moscowgentalman

Recovered, PR is now OK... @anoopsharma00 @DaveBirdsall @zellerh , please help to take a review

moscowgentalman avatar Apr 25 '18 03:04 moscowgentalman

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

Traf-Jenkins avatar Apr 25 '18 03:04 Traf-Jenkins

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

Traf-Jenkins avatar Apr 25 '18 05:04 Traf-Jenkins

This PR is not ready, far from ready, sorry, let me close it.

moscowgentalman avatar Apr 25 '18 12:04 moscowgentalman

reopen just for some tests, Do not merge

moscowgentalman avatar Apr 25 '18 16:04 moscowgentalman

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

Traf-Jenkins avatar Apr 25 '18 16:04 Traf-Jenkins

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

Traf-Jenkins avatar Apr 25 '18 17:04 Traf-Jenkins

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

Traf-Jenkins avatar Apr 25 '18 17:04 Traf-Jenkins

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

Traf-Jenkins avatar Apr 25 '18 20:04 Traf-Jenkins

jenkins, retest

moscowgentalman avatar May 08 '18 01:05 moscowgentalman

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

Traf-Jenkins avatar May 08 '18 02:05 Traf-Jenkins

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

Traf-Jenkins avatar May 08 '18 05:05 Traf-Jenkins

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

Traf-Jenkins avatar May 08 '18 06:05 Traf-Jenkins

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

Traf-Jenkins avatar May 08 '18 09:05 Traf-Jenkins

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

Traf-Jenkins avatar May 09 '18 17:05 Traf-Jenkins

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

Traf-Jenkins avatar May 09 '18 19:05 Traf-Jenkins

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

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

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

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