Skip to content
New issue

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

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

Already on GitHub? # to your account

Add manual client setup to README, lift query & ingest to top-level and more #34

Merged
merged 12 commits into from
Nov 28, 2022

Conversation

bahlo
Copy link
Member

@bahlo bahlo commented Nov 24, 2022

  • Showcase manual client setup in README.md
  • Move query_apl to top-level query
  • Move query to top-level query_legacy
  • Prefix legacy query methods, remove apl prefix
  • Move ingest methods to top-level
  • Rename {ingest_raw => ingest_bytes}
  • Update examples structure and build them on CI

I recommend going commit-by-commit for the review.

Also fixes DX-382

@bahlo bahlo requested review from lukasmalkmus and dasfmi November 24, 2022 09:05
@bahlo bahlo force-pushed the arne/dx-369-add-manual-client-setup-to-axiom-rs-in branch 3 times, most recently from 097216e to c07449d Compare November 24, 2022 09:58
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@bahlo bahlo force-pushed the arne/dx-369-add-manual-client-setup-to-axiom-rs-in branch from e4df0e5 to 971a1bc Compare November 24, 2022 11:15
@bahlo bahlo force-pushed the arne/dx-369-add-manual-client-setup-to-axiom-rs-in branch from 971a1bc to 6dccd71 Compare November 24, 2022 11:16
This commit lifts the main.rs files up to examples/, where it can be
picked up by `cargo run --examples` or `cargo run --example xyz`.
This adds more context to errors and makes debugging easier.
@bahlo bahlo force-pushed the arne/dx-369-add-manual-client-setup-to-axiom-rs-in branch from 6dccd71 to 3910bba Compare November 24, 2022 11:30
@bahlo bahlo requested a review from lukasmalkmus November 24, 2022 11:32
Chrono deprecated the timestamp method, this switches to the
non-panicking timestamp_opts method.
@bahlo bahlo merged commit 046a750 into main Nov 28, 2022
@bahlo bahlo deleted the arne/dx-369-add-manual-client-setup-to-axiom-rs-in branch November 28, 2022 08:30
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants