Closed amindadgar closed 6 days ago
[!WARNING]
Rate limit exceeded
@amindadgar has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 6 minutes and 15 seconds before requesting another review.
⌛ How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.🚦 How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.📥 Commits
Reviewing files that changed from the base of the PR and between a96c9f82f0dc92880224c0e16e7b60501927ebf9 and 07d6ca79978f1cc61ea8681889c444bf3acb88d4.
The pull request modifies the ExtractMessages
, ExtractMessagesDaily
, and several related classes by changing the chat_id
parameter type from str
to int
in their constructors. Additionally, the ExtractMessagesDaily
class updates the logic for handling message timestamps, now treating them as seconds instead of milliseconds. Corresponding changes are made in the unit tests to reflect these updates. The create_telegram_dag
function also sees enhancements with the introduction of a msg_count
variable for improved logging of extracted messages. The overall focus is on data type consistency and clarity in message extraction processes.
File | Change Summary |
---|---|
dags/hivemind_etl_helpers/src/db/telegram/extract/messages.py |
Changed chat_id parameter type from str to int in ExtractMessages constructor. |
dags/hivemind_etl_helpers/src/db/telegram/extract/messages_daily.py |
Changed chat_id parameter type from str to int in ExtractMessagesDaily constructor; updated message date extraction logic to use seconds. |
dags/hivemind_etl_helpers/tests/unit/test_telegram_extract_daily_messages.py |
Updated chat_id from string to integer in TestExtractMessagesDaily ; adjusted timestamps in tests to reflect seconds instead of milliseconds. |
dags/hivemind_telegram_etl.py |
Introduced msg_count variable for logging extracted message counts; updated related logging statements. |
dags/analyzer_helper/telegram/extract_raw_data.py |
Changed chat_id parameter type from str to int in ExtractRawInfo constructor. |
dags/analyzer_helper/telegram/extract_raw_members.py |
Changed chat_id parameter type from str to int in ExtractRawMembers constructor. |
dags/analyzer_helper/telegram/tests/integration/test_telegram_extract_raw_members.py |
Updated chat_id from string to integer in TestExtractRawMembers class. |
dags/hivemind_etl_helpers/src/db/telegram/utils/platform.py |
Changed chat_id parameter type from str to int in TelegramPlatform constructor. |
ExtractMessagesDaily
class in messages_daily.py
has a similar change to the chat_id
parameter type, aligning with the changes made in the ExtractMessages
class in messages.py
.ExtractMessages
class to exclude banned users are related to the ExtractMessages
class's functionality, which is directly impacted by the changes in the main PR.ExtractMessagesDaily
class in the __init__.py
file indicates a direct relationship with the changes made to the ExtractMessages
class, as both classes are part of the same extraction functionality for Telegram messages.🐰 In the land of code, where messages flow,
A change inchat_id
helps data to grow.
From strings to integers, we make our way,
With clearer logs to brighten the day.
Timestamps in seconds, oh what a delight,
Our rabbit code hops, ready for flight! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Summary by CodeRabbit
New Features
ExtractMessages
,ExtractMessagesDaily
, and other related classes to acceptchat_id
as an integer, enhancing data type consistency.ExtractMessagesDaily
to directly use timestamps in seconds.Bug Fixes
Tests
chat_id
type and timestamp handling.