[SENTRY-1604] Sentry JSON message factory: Need more information in alter partition event

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.0
• Fix Version/s: 2.0.0
• Component/s: Sentry
• Labels: patch-available
• Assignee: Nachiket Vaidya
• Reporter: Nachiket Vaidya
• Created: 20/Jan/17 20:09
• Updated: 24/Jul/17 14:52
• Resolved: 08/Feb/17 20:19

Description

When the query is:

alter table test_b partition (bbb = '111', aaa='222', ccc='333') rename to partition (bbb = 'r1', aaa='r2', ccc='r3')

Update:

|  8375 |     8371 | 1469487373 | ALTER_PARTITION | test_a                                                                                          | test_b        | {"timestamp":1469487373,"servicePrincipal":"","server":"","table":"test_b","db":"test_a","values":{"111", "222", "333"},"eventType":"ALTER_PARTITION"}                                                                        

We need to provide more info in notification entry.

Currently, message factory for alter partition does not store new partition values. It just stores old values. For alter partition with partition change, we need both values.

In this fix, sentry message factory stores also new partition values in alter partition message in addition to old values. Added unit test cases for the same.

Added new field "newValues" to store new partition values in SentryJSONAlterPartitionMessage.

Issue Links

• links to: review # 56000