Skip to content

fix(activity): Use long as id for activity - #2166

Open
CarlSchwan wants to merge 1 commit into
masterfrom
bugfix/activity-id-long
Open

CarlSchwan wants to merge 1 commit into
masterfrom
bugfix/activity-id-long

Conversation

@CarlSchwan

Copy link
Copy Markdown
Member

This is already incorrect when there is more than 4.000.000 activity entries in an instance and get even more incorrect once activity are ported to snowflake ids.

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (N/A)

This is already incorrect when there is more than 4.000.000 activity entries in an instance and get even more incorrect
once activity are ported to snowflake ids.

Assisted-by: ClaudeCode:claude-sonnet-5-5
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

SpotBugs

CategoryBaseNew
Bad practice3131
Correctness2828
Dodgy code1313
Internationalization66
Malicious code vulnerability2626
Multithreaded correctness77
Performance33
Security33
Total117117

* with a fileId of -1 instead
*/
@Deprecated
public GetActivitiesRemoteOperation(int lastGiven) {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Better to break existing usage thus we can update client code?

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants