Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hotfix sync queries #192

Closed
wants to merge 2 commits into from
Closed

Hotfix sync queries #192

wants to merge 2 commits into from

Conversation

agy-why
Copy link
Member

@agy-why agy-why commented Jul 31, 2023

The order of the columns in case of SELECT * and sync queries were depending on the ingestion order of the columns (kind of random).

We correct that in ordering the columns according to the order metadata information.

@agy-why
Copy link
Member Author

agy-why commented Jul 31, 2023

bad tree

@agy-why agy-why closed this Jul 31, 2023
@agy-why agy-why deleted the hotfix-sync-queries branch July 31, 2023 10:34
@coveralls
Copy link

Pull Request Test Coverage Report for Build 5713521397

  • 9 of 26 (34.62%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.1%) to 64.116%

Changes Missing Coverage Covered Lines Changed/Added Lines %
daiquiri/query/process.py 0 3 0.0%
daiquiri/metadata/serializers/validators.py 4 18 22.22%
Totals Coverage Status
Change from base Build 5609252153: -0.1%
Covered Lines: 4717
Relevant Lines: 7357

💛 - Coveralls

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.

None yet

2 participants