Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix function_clause error in tracking_status/2 #3229

Merged
merged 1 commit into from
Jul 23, 2021

Conversation

ansd
Copy link
Member

@ansd ansd commented Jul 23, 2021

Before this commit:

> ./sbin/rabbitmq-streams stream_status --tracking s1
Status of stream s1 on node rabbit@localhost ...
Error:
{:function_clause, [{:rabbit_stream_queue, :"-tracking_status/2-fun-0-", [:offsets, %{"s1-1" => 5}, []], [file: 'src/rabbit_stream_queue.erl', line: 608]}, {:maps, :fold_1, 3, [file: 'maps.erl', line: 410]}, {:rabbit_stream_queue, :tracking_status, 2, []}]}

After this commit:

> ./sbin/rabbitmq-streams stream_status --tracking s1
Status of stream s1 on node rabbit@localhost ...
┌────────┬───────────┬───────┐
│ type   │ reference │ value │
├────────┼───────────┼───────┤
│ offset │ s1-1      │ 51    │
└────────┴───────────┴───────┘

rabbitmq-streams stream_status --tracking <stream> also fails on v3.9.0.
Needs to be backported to v3.9.x.

GitHub action failures seem to be unrelated?

Before this commit:

> ./sbin/rabbitmq-streams stream_status --tracking s1
Status of stream s1 on node rabbit@localhost ...
Error:
{:function_clause,
[{:rabbit_stream_queue, :"-tracking_status/2-fun-0-",
[:offsets, %{"s1-1" => 5}, []],
[file: 'src/rabbit_stream_queue.erl', line: 608]},
{:maps, :fold_1, 3, [file: 'maps.erl', line: 410]},
{:rabbit_stream_queue, :tracking_status, 2, []}]}

After this commit:

> ./sbin/rabbitmq-streams stream_status --tracking s1
Status of stream s1 on node rabbit@localhost ...
┌────────┬───────────┬───────┐
│ type   │ reference │ value │
├────────┼───────────┼───────┤
│ offset │ s1-1      │ 51    │
└────────┴───────────┴───────┘
@michaelklishin michaelklishin merged commit 23432f3 into master Jul 23, 2021
@michaelklishin michaelklishin deleted the stream-status-tracking branch July 23, 2021 18:08
@michaelklishin michaelklishin added this to the 3.9.1 milestone Jul 23, 2021
@gerhard
Copy link
Contributor

gerhard commented Jul 23, 2021

Great find & fix!

I am curious to know when this started failing. Was it before the 3.9.0 RCs or after? Maybe this bug was there even before the betas 🙇‍♂️

@ansd
Copy link
Member Author

ansd commented Jul 23, 2021

The bug was there before the betas.

michaelklishin added a commit that referenced this pull request Jul 23, 2021
Fix function_clause error in tracking_status/2

(cherry picked from commit 23432f3)
@michaelklishin
Copy link
Member

Backported to v3.9.x.

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.

None yet

3 participants