vdiravka commented on a change in pull request #1559: DRILL-540: Allow querying 
hive views in Drill
URL: https://github.com/apache/drill/pull/1559#discussion_r243310862
 
 

 ##########
 File path: 
contrib/storage-hive/core/src/test/java/org/apache/drill/exec/impersonation/hive/TestStorageBasedHiveAuthorization.java
 ##########
 @@ -145,6 +148,41 @@ private static void 
setStorabaseBasedAuthorizationInHiveConf() {
     return hiveConfig;
   }
 
+  /**
+   * User       Groups
+   * <br/>
 
 Review comment:
   I will put a new comment, since the previous one is swallowed by your commit 
force-pushing.
   
   You didn't get an idea, I suggested to add `</br>` for every line of your 
table (use `ctrl+q` for checking it in Intelij).
   But please forgot about that, the proper way is to use `<table>`, `<tr>`, 
`<td>` tags. Please follow it.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to