parseable icon indicating copy to clipboard operation
parseable copied to clipboard

fix: modified populate_via_listing function to push first prefix entr…

Open shahidnasimkhan opened this issue 1 year ago • 2 comments

…y while creating minute_resolve hashmap (#692)

Fixes #692

Description

modified populate_via_listing function to push first prefix entry while creating minute_resolve hashmap


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.

shahidnasimkhan avatar Apr 09 '24 08:04 shahidnasimkhan

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

github-actions[bot] avatar Apr 09 '24 08:04 github-actions[bot]

I have read the CLA Document and I hereby sign the CLA

@shahidnasimkhan can you add this comment here.

nitisht avatar Apr 09 '24 10:04 nitisht

I have read the CLA Document and I hereby sign the CLA

shahidnasimkhan avatar Apr 10 '24 06:04 shahidnasimkhan

@shahidnasimkhan can you please provide the steps to reproduce the issue? is it like an edge case or a normal scenario? I am able to get the results as expected. Screenshot 2024-04-12 at 11 36 27 AM Screenshot 2024-04-12 at 11 37 12 AM Screenshot 2024-04-12 at 11 36 13 AM

nikhilsinhaparseable avatar Apr 12 '24 06:04 nikhilsinhaparseable

Screenshot from 2024-04-13 11-25-32

@nikhilsinhaparseable its not a corner case following is my post request from postman

{ "query": "select p_timestamp from sam", "startTime": "2024-04-05T07:19:00.000+00:00", "endTime": "2024-04-05T07:20:00.000+00:00" }

but it returns empty in response. you can see the records in attached screenshot above

shahidnasimkhan avatar Apr 13 '24 06:04 shahidnasimkhan