Skip to content

Conversation

rekby
Copy link
Member

@rekby rekby commented Aug 11, 2025

No description provided.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes author functionality from the issue management system by eliminating the TitleAuthor record class and simplifying the data model to work with issue titles only.

  • Removed the TitleAuthor record class entirely
  • Simplified saveAll method to accept a list of strings (titles) instead of TitleAuthor objects
  • Updated the database schema and queries to remove author-related fields

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
TitleAuthor.java Complete removal of the TitleAuthor record class
IssueYdbRepository.java Modified saveAll method to work with strings instead of TitleAuthor objects, removed author field from schema
Application.java Updated saveAll call to pass string titles directly instead of TitleAuthor instances
Comments suppressed due to low confidence (1)

dev-1/lesson-8.1/java/src/main/java/tech/ydb/app/IssueYdbRepository.java:76

  • The INSERT statement appears to be missing the author column from the table schema, but the table may still have this column defined elsewhere. This could cause SQL errors if the table schema hasn't been updated to remove the author column.
                        UPSERT INTO issues

@rekby rekby merged commit 070d58e into main Aug 18, 2025
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.

2 participants