Skip to content

Conversation

@satyaog
Copy link
Member

@satyaog satyaog commented Aug 22, 2025

No description provided.

@satyaog satyaog force-pushed the schema branch 2 times, most recently from 88a92cb to 4122d32 Compare August 22, 2025 22:06

# Use auto-incrementing integer as primary key
topic_id = Column(BigInteger, primary_key=True, autoincrement=True)
hashid = Column(LargeBinary, unique=True, nullable=True)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if we need the hashid anymore?

@satyaog
Copy link
Member Author

satyaog commented Aug 22, 2025

I've ported. the old schema with some modifications and clean-up but I might have removed too much. Let me know if that's the case. Then I'll add the interface to commit to the database with tests but I'm still reading the old code to understand how things works


paper_id = Column(ForeignKey("paper.paper_id"), primary_key=True, nullable=False)
key = Column(Text, primary_key=True, nullable=False)
update_key = Column(Text, nullable=True)
Copy link
Member Author

@satyaog satyaog Aug 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the use of update_key?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant