Sfoglia il codice sorgente

HUE-5145 [editor] Update top tables query to take database in parameter

Romain Rigaux 9 anni fa
parent
commit
d1875c7

+ 5 - 2
desktop/libs/metadata/src/metadata/optimizer_client.py

@@ -170,8 +170,11 @@ class OptimizerApi(object):
         raise PopupException(e, title=_('Error while accessing Optimizer'))
 
 
-  def top_tables(self, workfloadId=None):
-    return self._exec('get-top-tables', ['--tenant', self._product_name])
+  def top_tables(self, workfloadId=None, database_name='default'): 
+    return self._exec('get-top-tables', [
+        '--tenant', self._product_name,
+        '--db-name', database_name
+    ])
 
 
   def table_details(self, database_name, table_name):

+ 2 - 1
desktop/libs/metadata/src/metadata/optimizer_client_tests.py

@@ -108,7 +108,8 @@ class TestOptimizerApi(object):
 
 
   def test_top_tables(self):
-    resp = self.api.top_tables()
+    database_name = 'default'
+    resp = self.api.top_tables(database_name=database_name)
 
     assert_true(isinstance(resp['results'], list), resp) # No status code currently
 

+ 3 - 2
desktop/libs/metadata/src/metadata/urls.py

@@ -36,14 +36,15 @@ urlpatterns = patterns('metadata.navigator_api',
 
 # Optimizer API
 urlpatterns += patterns('metadata.optimizer_api',
-  url(r'^api/optimizer_api/top_tables/?$', 'top_tables', name='top_tables'),  
-  url(r'^api/optimizer_api/query_compatibility/?$', 'query_compatibility', name='query_compatibility'),
   url(r'^api/optimizer_api/upload_history/?$', 'upload_history', name='upload_history'),
   url(r'^api/optimizer_api/popular_values/?$', 'popular_values', name='popular_values'),
   url(r'^api/optimizer_api/similar_queries/?$', 'similar_queries', name='similar_queries'),
 
   #v2
   url(r'^api/optimizer_api/get_tenant/?$', 'get_tenant', name='get_tenant'),
+
+  url(r'^api/optimizer_api/top_tables/?$', 'top_tables', name='top_tables'),
   url(r'^api/optimizer_api/table_details/?$', 'table_details', name='table_details'),
   url(r'^api/optimizer_api/query_risk/?$', 'query_risk', name='query_risk'),
+  url(r'^api/optimizer_api/query_compatibility/?$', 'query_compatibility', name='query_compatibility'),
 )