Skip to content

Conversation

PacificViking
Copy link
Contributor

@PacificViking PacificViking commented Jul 28, 2025

Description

As title. Use official y-scope branches instead of personal ones.

Checklist

  • The PR satisfies the contribution guidelines.
  • This is a breaking change and that has been indicated in the PR title, OR this isn't a
    breaking change.
  • Necessary docs have been updated, OR no docs need to be updated.

Validation performed

@PacificViking PacificViking requested a review from a team as a code owner July 28, 2025 06:11
Copy link

coderabbitai bot commented Jul 28, 2025

Warning

Rate limit exceeded

@Bill-hbrhbr has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 10 minutes and 27 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 for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 78be436 and 37c88aa.

📒 Files selected for processing (3)
  • assets/presto_parquet/Dockerfile (2 hunks)
  • assets/presto_parquet/include/etc_worker/config.properties (1 hunks)
  • assets/presto_parquet/main.py (1 hunks)
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@PacificViking PacificViking changed the title Use yscope's github branch for presto+parquet. feat: Use yscope's github branch for presto+parquet. Jul 28, 2025
@PacificViking PacificViking force-pushed the john_yscope_presto_parquet branch from 5032431 to 65c4948 Compare July 28, 2025 19:15
@Bill-hbrhbr Bill-hbrhbr changed the title feat: Use yscope's github branch for presto+parquet. fix(presto_parquet): Use yscope's github branch for presto+parquet. Sep 24, 2025
@kirkrodrigues kirkrodrigues changed the title fix(presto_parquet): Use yscope's github branch for presto+parquet. fix(presto_parquet): Use yscope's github branch for Presto + Parquet. Sep 25, 2025
@kirkrodrigues kirkrodrigues changed the title fix(presto_parquet): Use yscope's github branch for Presto + Parquet. fix(presto_parquet): Use y-scope/presto for Presto + Parquet. Sep 25, 2025
Copy link
Member

@kirkrodrigues kirkrodrigues left a comment

Choose a reason for hiding this comment

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

Deferring to @Bill-hbrhbr's review.

@Bill-hbrhbr Bill-hbrhbr merged commit 5beffde into y-scope:main Sep 29, 2025
4 checks passed
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.

3 participants