Skip to content

Conversation

@trueleo
Copy link
Contributor

@trueleo trueleo commented Dec 27, 2022

Description

This PR adds ability for a stream to define altered schema with a dedicated timestamp column if there is none provided in the first request. This column is called p_datetime

This PR has:

  • been tested to ensure log ingestion and log query works.
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added documentation for new or modified features or behaviors.

This PR adds ability for a stream to define altered schema with
a dedicated timestamp column if there is none provided in the first
request. This column is called `p_datetime`
type LocalWriterGuard<'a> = MutexGuard<'a, Option<StreamWriter<std::fs::File>>>;

const DEFAULT_TIMESTAMP_KEY: &str = "p_timestamp";
const TIME_KEYS: &[&str] = &["time", "datetime", "timestamp"];
Copy link
Member

Choose a reason for hiding this comment

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

const TIME_KEYS: &[&str] = &["time", "datetime", "timestamp", “date”];

@nitisht nitisht merged commit 784c0aa into parseablehq:main Dec 28, 2022
@trueleo trueleo deleted the auto_add_datetime branch December 28, 2022 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants