Skip to content

Update and refactor XQueue documentation#1413

Open
sarina wants to merge 2 commits intomainfrom
sarina/external-graders-refactor
Open

Update and refactor XQueue documentation#1413
sarina wants to merge 2 commits intomainfrom
sarina/external-graders-refactor

Conversation

@sarina sarina requested review from UsamaSadiq and ormsbee March 24, 2026 17:58
Copy link
Contributor

@ormsbee ormsbee left a comment

Choose a reason for hiding this comment

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

@UsamaSadiq: One question for you regarding RabbitMQ vs. Redis.


### Step 4. Ensure Event Bus is Running

Tutor uses RabbitMQ by default.
Copy link
Contributor

Choose a reason for hiding this comment

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

@UsamaSadiq: Hasn't Tutor switched over to Redis for this?

Copy link
Member

@UsamaSadiq UsamaSadiq Mar 26, 2026

Choose a reason for hiding this comment

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

@ormsbee Good catch. I'm not familiar enough with current Tutor config but upon second look, yes Redis is being used by default now.
I had the old 2U notes in mind when reviewing this where both RabbitMQ or Redis were considered to be viable alternatives.

@sarina We would need to change the mention of RabbitMQ with Redis in the docs as per the latest standard.


### Step 4. Ensure Event Bus is Running

Tutor uses RabbitMQ by default.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Tutor uses RabbitMQ by default.
Tutor uses Redis by default.

tutor local ps
```

RabbitMQ should be listed.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
RabbitMQ should be listed.
Redis should be listed.

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