Skip to content

Conversation

@nikhilsinhaparseable
Copy link
Contributor

  1. fixed banner spacing
  2. modified server mode: All to Standalone, Ingest to Distributed (Ingest), Query to Distributed (Query)
  3. updated server mode in about API response
  4. updated logic for env var P_INGESTOR_URL to use HOSTNAME and PORT from env
  5. remvoed put cache api from querier
  6. added put cache api to ingester to enable caching

Fixes #XXXX.

Description


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.

1. fixed banner spacing
2. modified server mode: All to Standalone, Ingest to Distributed (Ingest), Query to Distributed (Query)
3. updated server mode in about API response
4. updated logic for env var P_INGESTOR_URL to use HOSTNAME and PORT from env
5. remvoed put cache api from querier
6. added put cache api to ingester
1. renamed ingester to ingestor
2. corrected cache flow for ingestors and standalone
3. removed query, other logstream apis for ingestors
@nitisht nitisht requested review from Eshanatnight and nitisht April 10, 2024 13:57
@nitisht nitisht merged commit 0d5f1fd into parseablehq:main Apr 10, 2024
nikhilsinhaparseable added a commit to nikhilsinhaparseable/parseable that referenced this pull request Apr 20, 2024
1. fixed banner spacing
2. modified server mode: All to Standalone, Ingest to Distributed (Ingest), Query to Distributed (Query)
3. updated server mode in about API response
4. updated logic for env var P_INGESTOR_URL to use HOSTNAME and PORT from env
5. remove put cache api from querier
6. added put cache api to ingestor
7. renamed ingester to ingestor
8. corrected cache flow for ingestors and standalone
9. removed query, other logstream apis for ingestors
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