FALCON-2047 Use /apps/falcon/extensions/mirroring as base dir for HiveDR instead of /apps/data-mirroring/ #252
Conversation
…eDR instead of /apps/data-mirroring/
|
+1 |
| <excludes> | ||
| <exclude>*/**</exclude> | ||
| </excludes> | ||
| <fileMode>0770</fileMode> |
There was a problem hiding this comment.
Can you please help me understand the reason for extra write permission here..
There was a problem hiding this comment.
@PraveenAdlakha : This has been the behavior since HiveDR feature was implemented.
Here is the reason why we falcon expects 770 perm for /apps/data-mirroring now /apps/extensions/mirroring directory :
Directory is used by multiple mirroring jobs to store replication specific data. Deleting this data, even accidentally, can cause mirror jobs to redo already replicated events and this can be expensive. Hence we decided that the directory has to be read+writable by Falcon and multiple users who belong to specific group, which in this case is “users" This way, rest of the world does not have permission to read/delete the data.
There was a problem hiding this comment.
thanks for the explanation @sowmyaramesh
|
+1 for the review @sowmyaramesh |
|
Looks good. +1 . |
|
+1 LGTM |
…eDR instead of /apps/data-mirroring/ Author: Sowmya Ramesh <sramesh@hortonworks.com> Reviewers: "Venkat Ranganathan <venkat@hortonworks.com>, Balu Vellanki <balu@apache.org>, Peeyush B <pbishnoi@hortonworks.com>, Praveen Adlakha <adlakha.praveen@gmail.com>" Closes apache#252 from sowmyaramesh/FALCON-2047
No description provided.