cgivre commented on a change in pull request #2332:
URL: https://github.com/apache/drill/pull/2332#discussion_r761251990



##########
File path: contrib/storage-phoenix/README.md
##########
@@ -0,0 +1,98 @@
+# [DRILL-7863](https://issues.apache.org/jira/browse/DRILL-7863): Add Storage 
Plugin for Apache Phoenix
+
+## Description
+
+ Phoenix say : "We put the SQL back in NoSQL",<br/>
+ Drill call : "We use the SQL to cross almost all the file systems and storage 
engines",<br/>
+ "Cheers !", users said.
+
+## Documentation
+

Review comment:
       @luocooong 
   Thank you so much for this submission.  I don't think I could add much more 
than @paul-rogers did for the code review.  
   I do have some suggestions for the docs however.  Could you please add a 
section about how to configure Drill to connect to Phoenix and perhaps a demo 
config?
   Thanks!




-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to