vvysotskyi commented on a change in pull request #2084:
URL: https://github.com/apache/drill/pull/2084#discussion_r444052723



##########
File path: 
contrib/storage-ipfs/src/main/java/org/apache/drill/exec/store/ipfs/IPFSContext.java
##########
@@ -0,0 +1,95 @@
+/*
+ * Copyright (c) 2018-2020 Bowen Ding, Yuedong Xu, Liang Wang

Review comment:
       Neither current code nor docs don't have any mention about their 
authors, and I don't think that it should be changed. As I mentioned above, it 
would be hard to keep a list of **all** contributors for the specific code 
part. Git history allows to see specific contribution, but just putting the 
author name to the code doesn't show anything related to the changes. What 
would we do with authorship if the plugin will be rewritten in the future from 
scratch, or significantly refactored, so it would differ significantly from its 
initial version? Using just git history solves this issue, and I don't think 
that we should introduce more complexity. 




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to