Skip to content
This repository has been archived by the owner on Sep 27, 2019. It is now read-only.

[15721]Sequence Final PR #1367

Closed
wants to merge 92 commits into from
Closed

[15721]Sequence Final PR #1367

wants to merge 92 commits into from

Conversation

HenryZhou0333
Copy link

@HenryZhou0333 HenryZhou0333 commented May 15, 2018 •
edited
Loading

Part of our changes are based on temp table team.

HenryZhou0333 and others added 30 commits March 22, 2018 16:07
@HenryZhou0333
finish part of parser; need to add handler function in create_executo…
30d48ac 
…r.cpp
@HenryZhou0333
finish part of parser; need to add handler function in create_executo…
086b493 
…r.cpp
@HenryZhou0333
Merge branch 'master' of https://github.com/HenryZhou0333/peloton
59b85f6
@HenryZhou0333
Merge remote-tracking branch 'upstream/master'
282afb3
@HenryZhou0333
add postgresql parser test for sequences
28b702e
@HenryZhou0333
remove AS in sequencce
ddb3dd7
@HenryZhou0333
add exception for redefined parser
26230e3
@HenryZhou0333
finish compiling create sequence
b273056
@HenryZhou0333
Merge remote-tracking branch 'upstream/master'
d90a649
@HenryZhou0333
having issue with GetSequence()
c1a4e1d
@HenryZhou0333
add new GetSequence impl; fix BasicTest in sequence
f8ec42a
@HenryZhou0333
add tests for sequence
0d3521c
@HenryZhou0333
fix merge conflict
8fe7d6c
@HenryZhou0333
one step before submitting pr
87680be
@HenryZhou0333
change way of update pg_sequence table in nextval
7d7955c
style fix
503c89f
@HenryZhou0333
rename sequence test
7762354
@HenryZhou0333
Merge remote-tracking branch 'upstream/master'
b340c86
@HenryZhou0333
remove unnecessary file
6897566
@danae-s
adding nextval & currval functions; comment out lock and nextval not …
b8c7317 
…functioning well
@HenryZhou0333
update pg_sequence after calling nextval; have seg fault in txn;
d665c94
@HenryZhou0333
temporary add code from other team
e71a21c
@HenryZhou0333
fix conflict with master
ca8f8fe
@HenryZhou0333
again fix conflicts
85af574
Implemented basic func for temp
2649a7d
@HenryZhou0333
move sequence catalog into system catalogs
7997a9a
Make files compiled
dadb91f
fix bug in setting session namespace
15913b6
added drop temp schema
0e97cb9
@HenryZhou0333
fix issues in comments; add sanity check for sequence arguments
35d6a5a
yijiacui and others added 25 commits May 13, 2018 04:02
finished foreign key feature
747e89a
added oid_table for txn. TODO: on commit support for create table
fde290f
Added parser option for on commit upon creating table
97c5930
@lym953
Fix compilation error
03f0729
@lym953
Fix a bug in ForeignKeyTest
f318be8
@lym953
Simplify test code using EXPECT_THROW
a073cf0
@lym953
Fix bugs in PsqlTempTableTests
7389bf1
added on commit drop option
b31e2ee
Merge branch 'temp_table' of https://github.com/lym953/peloton into t…
e632496 
…emp_table
@lym953
Enclose each test in PsqlTempTableTests with a try-catch block
535ee0d
@lym953
Add OnCommitOptionsTest for temp table
f3b0b33
@bdengNE
add drop seq tests and format the codesbased on review
b482c2e
@lym953
Comment out test code for ON COMMIT DELETE ROWS because it is not sup…
9ba0eea 
…ported now
@danae-s
Add tests.
60e222d
@HenryZhou0333
fix sequence_functions_test
b4dfb68
@danae-s
Fix tests.
9c4e2f3
@lym953
Remove TempTableTest.java
df186bb
fix bugs in foreign key constraint
7e49b44
Merge branch 'temp_table' of https://github.com/lym953/peloton into t…
bbc1193 
…emp_table
@danae-s
Build with release, remove unused variables.
c35ab86
@danae-s
Conflicts resolved
f282909
@danae-s
Fix tests. catalog table 8->9
0e61e12
@HenryZhou0333
fix test in valgrind
acac8fc
@HenryZhou0333
fix comment in pr2
65318ca
@HenryZhou0333
fix test
d85e1c7
@HenryZhou0333 HenryZhou0333 changed the title Sequence Final PR [15721]Sequence Final PR May 15, 2018
@apavlo
Copy link
Member

apavlo commented Jun 21, 2018

Replaced by #1407

@apavlo apavlo closed this Jun 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
class-project do not merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@HenryZhou0333 @apavlo @saatviks @danae-s @lym953 @bdengNE