Skip to content

Conversation

trueleo
Copy link
Contributor

@trueleo trueleo commented Aug 27, 2022

Fixes #51.

Description

Fixes logstream listing by using common prefixes.

This lists all the root level objects which in case of parseable are logstream directories.


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.

nitisht
nitisht previously approved these changes Aug 27, 2022
Copy link
Member

@nitisht nitisht left a comment

Choose a reason for hiding this comment

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

Great find!

@nitisht nitisht mentioned this pull request Aug 27, 2022
@nitisht nitisht merged commit 8ec038c into parseablehq:main Aug 27, 2022
@trueleo trueleo deleted the list_fix branch August 27, 2022 15:20
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.

New log streams not showing up in log streams list

2 participants