Skip to content

Conversation

dippindots
Copy link
Collaborator

@dippindots dippindots commented Oct 14, 2025

@dippindots @haynescd if i understand correctly, CH doesn't support this this function and it actually clear what it was doing in the first place in the absnce of group by? @haynescd this was causing it auth to fail.

@dippindots dippindots changed the base branch from master to demo-clickhouse-only-db October 14, 2025 20:10
@dippindots dippindots requested a review from alisman October 14, 2025 20:14
@dippindots dippindots added the bug label Oct 14, 2025
@dippindots dippindots self-assigned this Oct 14, 2025
@dippindots dippindots marked this pull request as ready for review October 16, 2025 02:58
@alisman alisman requested a review from haynescd October 16, 2025 14:49
Copy link
Collaborator

@haynescd haynescd left a comment

Choose a reason for hiding this comment

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

Makes sense.

ANY_VALUE(sample_list.category) AS "${prefix}category",
ANY_VALUE(sample_list.name) AS "${prefix}name",
ANY_VALUE(sample_list.description) AS "${prefix}description"
sample_list.cancer_study_id AS "${prefix}cancerStudyId",
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove the prefix as well... it is confusing and is set to "" anyways

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