Could you git add the test file too?

And push to an upstream branch

If headers still needed, to move up to a class variable

Do we really need all these headers?
Does it work with {}?
With only 'Content-Type': 'application/json',

Or/and 'X-Requested-With': 'XMLHttpRequest',

      'X-Requested-By': 'das',`?

Factor out the DAS url in a new config property in [jobbrowser]?
Note: will have to add extra properties for the authentication at some point.

To move up with the same imports.
Actually, we don't use TestCase so to remove and do like class TestHiveQueryApi():, not call to super, like below

Does it still fail afterwards?

Here we should test the API (i.e. apps(), app(), what we are implementing in the API), not the model (similar to below)

Small changes, can be done on a 3rd commit on top of the previous two, then we can ship and iterate more

We can't change this file without breaking all the other Job Browsers (so to workaround in the hive api module)

HEADERS = {'X-Requested-By': 'das'}

100 currently I guess instead, we can adapt when we take the filtering into account

As we will add more params (credentials etc..) soon, could we make it a subsection instead of single config params?
e.g. like https://github.com/cloudera/hue/blob/master/desktop/libs/metadata/src/metadata/conf.py#L276

if interface == 'queries-hive':

....

if interface == 'queries-hive':

  app_id = json.loads(request.body)['queryId'] 

else:

  app_id = json.loads(request.POST.get('app_id')

if interface == 'queries-hive':

  return JsonResponse(response['app'])

else:

  return JsonResponse(response)

Almost!

DAS --> QUERY_STORE
everywhere

Credentials for query store API.

URL of Query Store server.

Let's avoid DAS as the name and keep it more generic.
How about query_store like for the DAS db config?

nit: let's avoid the extra indentation

response['logs'] = get_api(request.user, interface, cluster=cluster).logs(

  app_id, app_type, log_name, json.loads(request.GET.get('is_embeddable', 'false').lower())

)

app_filters = dict([

    (key, value) for _filter in json.loads(request.POST.get('app_filters', '[]'))

    for key, value in list(_filter.items()) if value

])

])

extra space

SERVER --> API server

ditto

-2 indentation
Space after #
SERVER --> API server