Skip to content

Docs impr, smaller renamings, raw topic fix, mem leak fix #45

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 20 commits into from
Feb 28, 2023
Merged

Docs impr, smaller renamings, raw topic fix, mem leak fix #45

merged 20 commits into from
Feb 28, 2023

Conversation

@peter-quix
Copy link
Contributor

@peter-quix peter-quix commented Feb 24, 2023 •
edited
Loading

  • Add version migration
  • Fix RawTopic not using Get as the verb instead of Create
  • Fix old naming left in new code
  • DataFrame raised with 'timestamp' rather than 'time' (Consistency for Timestamp name #6 )
  • Fixed raw topic read
    • Fixed a typo (KafkaDataTime -> KafkaDateTime) in metadata
  • Changed key to byte in transport layer and raw read/write
  • Fixed some double dispose and memory leak

@peter-quix peter-quix marked this pull request as ready for review February 24, 2023 17:44
@peter-quix peter-quix changed the title Documentation improvements and smaller renamings Docs impr, smaller renamings, raw topic fix, mem leak fix Feb 27, 2023
@peter-quix peter-quix mentioned this pull request Feb 27, 2023
Closed
JotaBlanco
JotaBlanco previously approved these changes Feb 28, 2023
View reviewed changes
@peter-quix peter-quix merged commit 91a271f into main Feb 28, 2023
@peter-quix peter-quix deleted the feature/native-improvements-6 branch February 28, 2023 21:29
@peter-quix peter-quix mentioned this pull request Mar 2, 2023
Merged
tim-quix pushed a commit that referenced this pull request Nov 14, 2023
@srosam
[task/31427] - Fix links (#45)
22ada7d 
* correct all page paths

* remove space

* test

* test

* test

* Links adjustment and move sentiment analysis media

* image url fix

* Moved tutorial images closer to each tutorial

* api images

* Updates to links and file paths

* Links

* Index links

* Homepage links

* links

* remove junk

* Build prod with  --no-directory-urls

* updates

* Fix some paths

* Fix last link ;-)

* spelling fix

* PR Fixes

* Go one one level less

* definitions link fixes

* more changes :-(

* fixes

* ImageProcessing URL fixes

* last fix?

* Fixes

* add sdk docs back

Co-authored-by: Steve R <steve@quix.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@JotaBlanco JotaBlanco JotaBlanco approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@peter-quix @JotaBlanco @mikerosam