mneethiraj commented on code in PR #454:
URL: https://github.com/apache/atlas/pull/454#discussion_r2373573163


##########
graphdb/janusgraph-rdbms/src/main/resources/META-INF/postgres/create_schema.sql:
##########
@@ -25,12 +25,12 @@ CREATE INDEX IF NOT EXISTS 
atlas_entity_audit_idx_entity_id_event_time ON atlas_
 
 
 -- DB objects for JanusGraph backend store
-CREATE SEQUENCE IF NOT EXISTS janus_store_seq;
-CREATE SEQUENCE IF NOT EXISTS janus_key_seq;
-CREATE SEQUENCE IF NOT EXISTS janus_column_seq ;
+CREATE SEQUENCE IF NOT EXISTS janus_store_seq INCREMENT BY 1 CACHE 1;
+CREATE SEQUENCE IF NOT EXISTS janus_key_seq INCREMENT BY 1 CACHE 1000;
+CREATE SEQUENCE IF NOT EXISTS janus_column_seq INCREMENT BY 1 CACHE 1000;
 
-CREATE TABLE IF NOT EXISTS janus_store(id BIGINT DEFAULT 
NEXTVAL('janus_store_seq'::regclass), name VARCHAR(255) NOT NULL, PRIMARY 
KEY(id));
-CREATE TABLE IF NOT EXISTS janus_key(id BIGINT DEFAULT 
NEXTVAL('janus_key_seq'::regclass), store_id BIGINT NOT NULL, name BYTEA NOT 
NULL, PRIMARY KEY(id));
+CREATE TABLE IF NOT EXISTS janus_store(id BIGINT, name VARCHAR(255) NOT NULL, 
PRIMARY KEY(id));
+CREATE TABLE IF NOT EXISTS janus_key(id BIGINT, store_id BIGINT NOT NULL, name 
BYTEA NOT NULL, PRIMARY KEY(id));

Review Comment:
   Call to generate `id` is handled by JPA (see JanusKey.java), hence there is 
no need for `DEFAULT` value here.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to