search

data-dot-all / dataall

A modern data marketplace that makes collaboration among diverse users (like business, analysts and engineers) easier, increasing efficiency and agility in data projects on AWS.
https://data-dot-all.github.io/dataall/
Apache License 2.0
228 stars 82 forks source link

Investigate whether search lambda (search_handler.py ) can be combined with api_handler.py #1302

Open TejasRGitHub opened 3 months ago

TejasRGitHub commented 3 months ago

Is your idea related to a problem? Please describe. After this potential change - https://github.com/data-dot-all/dataall/pull/1236/. The search_handler and the api_handler have a lot of common code.

Describe the solution you'd like Combine search_handler.py and api_handler.py without losing any performance. This will reduce the use of one lambda. Throttling should be checked as in this case both gql and the catalog search request would be coming on a single lambda

P.S. Don't attach files. Please, prefer add code snippets directly in the message body.

dlpzx commented 3 months ago

Absolutely, thanks @TejasRGitHub for opening the issue. I think we shall consider it for v2.7 :)