Skip to content

Commit

Permalink
Remove processor and yaml changes referring to group_first_release_id (
Browse files Browse the repository at this point in the history
  • Loading branch information
snigdhas authored Jun 20, 2024
1 parent 19da4fe commit 7ec66c7
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ schema:
{ name: group_status, type: UInt, args: { size: 8 } },
{ name: group_substatus, type: UInt, args: { size: 8, schema_modifiers: [ nullable ] } },
{ name: group_priority, type: UInt, args: { size: 8, schema_modifiers: [ nullable ] } },
{ name: group_first_release_id, type: UUID, args: { schema_modifiers: [ nullable ] } },
{ name: group_first_seen, type: DateTime },
{ name: group_num_comments, type: UInt, args: { size: 64 } },

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ schema:
{ name: group_status, type: UInt, args: { size: 8 } },
{ name: group_substatus, type: UInt, args: { size: 8, schema_modifiers: [ nullable ] } },
{ name: group_priority, type: UInt, args: { size: 8, schema_modifiers: [ nullable ] } },
{ name: group_first_release_id, type: UUID, args: { schema_modifiers: [ nullable ] } },
{ name: group_first_seen, type: DateTime },
{ name: group_num_comments, type: UInt, args: { size: 64 } },

Expand Down Expand Up @@ -59,9 +58,6 @@ allocation_policies:
query_processors:
- processor: TableRateLimit
- processor: ConsistencyEnforcerProcessor
- processor: UUIDColumnProcessor
args:
columns: [group_first_release_id]

mandatory_condition_checkers:
- condition: ProjectIdEnforcer
Expand Down
1 change: 0 additions & 1 deletion snuba/datasets/processors/group_attributes_processor.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ def process_message(
"group_status": message["status"],
"group_substatus": message["substatus"],
"group_priority": message.get("priority", None),
"group_first_release_id": message.get("first_release_id", None),
"group_first_seen": datetime.strptime(
message["first_seen"], settings.PAYLOAD_DATETIME_FORMAT
),
Expand Down
5 changes: 0 additions & 5 deletions tests/datasets/test_group_attributes_processor.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import uuid
from datetime import datetime
from typing import Optional

Expand All @@ -15,8 +14,6 @@
from snuba.processor import ProcessedMessage
from snuba.writer import WriterTableRow

RELEASE_ID = uuid.uuid4()


@pytest.fixture
def group_created() -> GroupAttributesSnapshot:
Expand All @@ -27,7 +24,6 @@ def group_created() -> GroupAttributesSnapshot:
"status": 0,
"substatus": 7,
"priority": 25,
"first_release_id": RELEASE_ID,
"first_seen": "2023-02-27T15:40:12.223000Z",
"num_comments": 0,
"assignee_user_id": None,
Expand Down Expand Up @@ -65,7 +61,6 @@ def test_group_created(self, group_created):
"group_status": 0,
"group_substatus": 7,
"group_priority": 25,
"group_first_release_id": RELEASE_ID,
"group_first_seen": datetime.strptime(
group_created["first_seen"], settings.PAYLOAD_DATETIME_FORMAT
),
Expand Down

0 comments on commit 7ec66c7

Please sign in to comment.