Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/10085 )
Change subject: IMPALA-6749: Implement ALTER DATABASE SET DBPROPERTIES ...................................................................... Patch Set 4: I agree with some comments but not with others. I don't think the fear of abuse is rooted in any evidence. I have yet to see a single user that has seriously abused the TBLPROPERTIES mechanism. Really, we are the only ones abusing these properties, but we'll continue to do that regardless of whether we have this ALTER DATABASE or not. That said, I think it's fair to say that there's no real use case for this feature today, so it might not be worth the added complexity. -- To view, visit http://gerrit.cloudera.org:8080/10085 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I08bc4a9c6fa95dd135274fa1081bf24ebe14c99a Gerrit-Change-Number: 10085 Gerrit-PatchSet: 4 Gerrit-Owner: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]> Gerrit-Comment-Date: Wed, 18 Apr 2018 05:23:09 +0000 Gerrit-HasComments: No
