Skip to content

Replace string literals with constants for worker table column names #308

@JatinSanghvi

Description

@JatinSanghvi

This issue tracks the fix to address the code review comment in #226.

I know you said you didn't use constants because of code readability, but I don't like the potential risk of regression here by having these values be separate. So I'd strongly prefer that we make constants for each of these - even if it does decrease the readability of the query slightly.

Originally posted by @Charles-Gagnon in #226 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions