From fa50c42a214e56587fc6c0a0b428d3db75adb65e Mon Sep 17 00:00:00 2001 From: Muhamad Tohir Date: Thu, 16 Sep 2021 16:50:52 +0700 Subject: [PATCH 01/12] feat: init python client and store Co-Authored-By: Rahmat Hidayat --- clients/python/src/odpf/__init__.py | 0 clients/python/src/odpf/stencil.py | 10 ++++++++++ clients/python/src/odpf/store.py | 21 +++++++++++++++++++++ 3 files changed, 31 insertions(+) create mode 100644 clients/python/src/odpf/__init__.py create mode 100644 clients/python/src/odpf/stencil.py create mode 100644 clients/python/src/odpf/store.py diff --git a/clients/python/src/odpf/__init__.py b/clients/python/src/odpf/__init__.py new file mode 100644 index 00000000..e69de29b diff --git a/clients/python/src/odpf/stencil.py b/clients/python/src/odpf/stencil.py new file mode 100644 index 00000000..47afbebb --- /dev/null +++ b/clients/python/src/odpf/stencil.py @@ -0,0 +1,10 @@ +class MultiUrlClient: + def __init__(self, urls:list) -> None: + pass + + + + +class Client(MultiUrlClient): + def __init__(self, url: str) -> None: + super().__init__([url]) \ No newline at end of file diff --git a/clients/python/src/odpf/store.py b/clients/python/src/odpf/store.py new file mode 100644 index 00000000..cd4010c3 --- /dev/null +++ b/clients/python/src/odpf/store.py @@ -0,0 +1,21 @@ +import requests +from google.protobuf.descriptor_pb2 import FileDescriptorSet +from google.protobuf.descriptor import Descriptor +from google.protobuf.message_factory import GetMessages + +class Store: + def __init__(self): + self.data = {} + + def get(self, name) -> Descriptor: + return self.data.get(name) + + def load(self, url): + result = requests.get(url) + fds = FileDescriptorSet.FromString(result.text) + messages = GetMessages([file for file in fds.file]) + self.data.update(messages) + + + + \ No newline at end of file From 3f8f4f6bde3f46e687f74bcabbacfa13c93ae6b7 Mon Sep 17 00:00:00 2001 From: Muhamad Tohir Date: Thu, 16 Sep 2021 17:17:10 +0700 Subject: [PATCH 02/12] feat: setup client and message factory Co-Authored-By: Rahmat Hidayat --- clients/python/src/odpf/stencil.py | 30 ++++++++++++++++++++++++++---- clients/python/src/odpf/store.py | 5 +++-- 2 files changed, 29 insertions(+), 6 deletions(-) diff --git a/clients/python/src/odpf/stencil.py b/clients/python/src/odpf/stencil.py index 47afbebb..31d7c032 100644 --- a/clients/python/src/odpf/stencil.py +++ b/clients/python/src/odpf/stencil.py @@ -1,10 +1,32 @@ +from clients.python.src.odpf import store +from schedule import Scheduler +from google.protobuf.message import Message + +from store import Store class MultiUrlClient: - def __init__(self, urls:list) -> None: - pass - - + def __init__(self, urls:list, interval=3600, auto_refresh=False) -> None: + self._store = Store() + self._urls = urls + self._interval = interval + self._auto_refresh = auto_refresh + self._init_data() + self._schduler = Scheduler() + if self._auto_refresh: + self._schduler.every(self._interval).seconds.do(self.refresh) + #TODO: check whether scheduler executed immediatelly or not + self.refresh() + def refresh(self): + for url in self._urls: + self._store.load(url) + def get_descriptor(self, name:str) -> Message: + return self._store.get(name) + + def parse(self, name:str, data:str): + msg = self.get_descriptor(name) + return msg.ParseFromString(data) + class Client(MultiUrlClient): def __init__(self, url: str) -> None: super().__init__([url]) \ No newline at end of file diff --git a/clients/python/src/odpf/store.py b/clients/python/src/odpf/store.py index cd4010c3..e03f358c 100644 --- a/clients/python/src/odpf/store.py +++ b/clients/python/src/odpf/store.py @@ -1,13 +1,14 @@ +from google.protobuf.message import Message import requests from google.protobuf.descriptor_pb2 import FileDescriptorSet -from google.protobuf.descriptor import Descriptor +from google.protobuf.message import Message from google.protobuf.message_factory import GetMessages class Store: def __init__(self): self.data = {} - def get(self, name) -> Descriptor: + def get(self, name) -> Message: return self.data.get(name) def load(self, url): From f64a89d4d510206afeba7f987677e1358d32a6d7 Mon Sep 17 00:00:00 2001 From: "rahmat.hd" Date: Fri, 17 Sep 2021 15:20:31 +0700 Subject: [PATCH 03/12] fix: use streaming to download file desriptor set --- clients/python/src/odpf/stencil.py | 4 +--- clients/python/src/odpf/store.py | 8 ++------ 2 files changed, 3 insertions(+), 9 deletions(-) diff --git a/clients/python/src/odpf/stencil.py b/clients/python/src/odpf/stencil.py index 31d7c032..1b98f9ea 100644 --- a/clients/python/src/odpf/stencil.py +++ b/clients/python/src/odpf/stencil.py @@ -1,15 +1,13 @@ -from clients.python.src.odpf import store +from .store import Store from schedule import Scheduler from google.protobuf.message import Message -from store import Store class MultiUrlClient: def __init__(self, urls:list, interval=3600, auto_refresh=False) -> None: self._store = Store() self._urls = urls self._interval = interval self._auto_refresh = auto_refresh - self._init_data() self._schduler = Scheduler() if self._auto_refresh: self._schduler.every(self._interval).seconds.do(self.refresh) diff --git a/clients/python/src/odpf/store.py b/clients/python/src/odpf/store.py index e03f358c..8f62b046 100644 --- a/clients/python/src/odpf/store.py +++ b/clients/python/src/odpf/store.py @@ -12,11 +12,7 @@ def get(self, name) -> Message: return self.data.get(name) def load(self, url): - result = requests.get(url) - fds = FileDescriptorSet.FromString(result.text) + result = requests.get(url, stream=True) + fds = FileDescriptorSet.FromString(result.raw.read()) messages = GetMessages([file for file in fds.file]) self.data.update(messages) - - - - \ No newline at end of file From a6c2eb5a9c8a1bde9a8de85a76fae4627ef423ff Mon Sep 17 00:00:00 2001 From: Muhamad Tohir Date: Sat, 18 Sep 2021 09:07:19 +0700 Subject: [PATCH 04/12] fix: add requirements.txt --- clients/python/requirements.txt | 6 ++++++ 1 file changed, 6 insertions(+) create mode 100644 clients/python/requirements.txt diff --git a/clients/python/requirements.txt b/clients/python/requirements.txt new file mode 100644 index 00000000..5db6db97 --- /dev/null +++ b/clients/python/requirements.txt @@ -0,0 +1,6 @@ +protobuf==3.17.3 +pytest==6.0.1 +pytest-cov==2.12.1 +mock==4.0.3 +schedule==1.1.0 +requests==2.26.0 \ No newline at end of file From 0adc79dce05f31c34447eb692aa9624ecb6491ab Mon Sep 17 00:00:00 2001 From: Muhamad Tohir Date: Sat, 18 Sep 2021 09:08:17 +0700 Subject: [PATCH 05/12] feat: support load from url and bytes --- clients/python/src/odpf/stencil.py | 2 +- clients/python/src/odpf/store.py | 9 +++++++-- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/clients/python/src/odpf/stencil.py b/clients/python/src/odpf/stencil.py index 1b98f9ea..a3d5d5b3 100644 --- a/clients/python/src/odpf/stencil.py +++ b/clients/python/src/odpf/stencil.py @@ -16,7 +16,7 @@ def __init__(self, urls:list, interval=3600, auto_refresh=False) -> None: def refresh(self): for url in self._urls: - self._store.load(url) + self._store.load(url=url) def get_descriptor(self, name:str) -> Message: return self._store.get(name) diff --git a/clients/python/src/odpf/store.py b/clients/python/src/odpf/store.py index 8f62b046..36e74261 100644 --- a/clients/python/src/odpf/store.py +++ b/clients/python/src/odpf/store.py @@ -11,8 +11,13 @@ def __init__(self): def get(self, name) -> Message: return self.data.get(name) - def load(self, url): + def _load_from_url(self, url): result = requests.get(url, stream=True) - fds = FileDescriptorSet.FromString(result.raw.read()) + return result.raw.read() + + def load(self, url:str=None, data:bytes=None): + if url: + data = self._load_from_url(url) + fds = FileDescriptorSet.FromString(data) messages = GetMessages([file for file in fds.file]) self.data.update(messages) From a9c2fd8d48569fb00ca020f1121e92b3cc1e8525 Mon Sep 17 00:00:00 2001 From: Muhamad Tohir Date: Sat, 18 Sep 2021 09:10:19 +0700 Subject: [PATCH 06/12] test: add test for client and store --- clients/python/conftest.py | 14 ++++++++++++++ clients/python/test/data/one.proto | 24 ++++++++++++++++++++++++ clients/python/test/test_client.py | 26 ++++++++++++++++++++++++++ 3 files changed, 64 insertions(+) create mode 100644 clients/python/conftest.py create mode 100644 clients/python/test/data/one.proto create mode 100644 clients/python/test/test_client.py diff --git a/clients/python/conftest.py b/clients/python/conftest.py new file mode 100644 index 00000000..b1948e36 --- /dev/null +++ b/clients/python/conftest.py @@ -0,0 +1,14 @@ +from subprocess import run + +import pytest +import os + +@pytest.fixture(scope="session") +def protoc_setup(): + print('Do protoc') + current_dir = os.path.dirname(os.path.realpath(__file__)) + output_file = os.path.join(current_dir, 'test/data/one.desc') + + input_dir = os.path.join(current_dir, 'test/data') + run(['protoc',f'--descriptor_set_out={output_file}','--include_imports',f'--proto_path={input_dir}','one.proto'], cwd=current_dir) + \ No newline at end of file diff --git a/clients/python/test/data/one.proto b/clients/python/test/data/one.proto new file mode 100644 index 00000000..90d58b0b --- /dev/null +++ b/clients/python/test/data/one.proto @@ -0,0 +1,24 @@ +syntax = "proto3"; + +package test; +import "google/protobuf/timestamp.proto"; +option java_package = "test.stencil"; + +message One { + int64 field_one = 1; +} + +message Two { + message Three { + string data = 1; + google.protobuf.Timestamp timestamp = 3; + } + Three id = 1; + message Four { + Four recursive = 2; + string field_two = 3; + message Five { + double id = 1; + } + } +} diff --git a/clients/python/test/test_client.py b/clients/python/test/test_client.py new file mode 100644 index 00000000..21f6e40f --- /dev/null +++ b/clients/python/test/test_client.py @@ -0,0 +1,26 @@ +from src.odpf.stencil import Client +from src.odpf.store import Store +from mock import patch + +URL = 'http://stencil.test/proto-descriptors/test/latest' +def get_file_desc(): + with open('test/data/one.desc', 'rb') as myfile: + desc = myfile.read() + return desc + +def test_store(protoc_setup): + file_desc = get_file_desc() + store = Store() + store.load(data=file_desc) + assert 'test.One' in store.data + assert isinstance(store.get('test.One'), store.get('test.One').__class__) + +@patch.object(Store, '_load_from_url') +def test_client(test_desc_from_url, protoc_setup): + file_desc = get_file_desc() + test_desc_from_url.return_value = file_desc + + client = Client(URL) + + assert client.get_descriptor('test.One') + assert client.get_descriptor('test.faile') is None From 2f75c63df7abd0785ef9960ea9d94b8fc9b72488 Mon Sep 17 00:00:00 2001 From: Muhamad Tohir Date: Sat, 18 Sep 2021 09:37:13 +0700 Subject: [PATCH 07/12] docs: add readme, license, .gitignore --- clients/python/.gitignore | 141 ++++++++++++++++++++++++++ clients/python/LICENSE | 201 ++++++++++++++++++++++++++++++++++++++ clients/python/README.md | 69 +++++++++++++ 3 files changed, 411 insertions(+) create mode 100644 clients/python/.gitignore create mode 100644 clients/python/LICENSE create mode 100644 clients/python/README.md diff --git a/clients/python/.gitignore b/clients/python/.gitignore new file mode 100644 index 00000000..a42d994c --- /dev/null +++ b/clients/python/.gitignore @@ -0,0 +1,141 @@ +# Byte-compiled / optimized / DLL files +__pycache__/ +*.py[cod] +*$py.class + +# C extensions +*.so + +# Distribution / packaging +.Python +build/ +develop-eggs/ +dist/ +downloads/ +eggs/ +.eggs/ +lib/ +lib64/ +parts/ +sdist/ +var/ +wheels/ +share/python-wheels/ +*.egg-info/ +.installed.cfg +*.egg +MANIFEST + +# PyInstaller +# Usually these files are written by a python script from a template +# before PyInstaller builds the exe, so as to inject date/other infos into it. +*.manifest +*.spec + +# Installer logs +pip-log.txt +pip-delete-this-directory.txt + +# Unit test / coverage reports +htmlcov/ +.tox/ +.nox/ +.coverage +.coverage.* +.cache +nosetests.xml +coverage.xml +*.cover +*.py,cover +.hypothesis/ +.pytest_cache/ +cover/ + +# Translations +*.mo +*.pot + +# Django stuff: +*.log +local_settings.py +db.sqlite3 +db.sqlite3-journal + +# Flask stuff: +instance/ +.webassets-cache + +# Scrapy stuff: +.scrapy + +# Sphinx documentation +docs/_build/ + +# PyBuilder +.pybuilder/ +target/ + +# Jupyter Notebook +.ipynb_checkpoints + +# IPython +profile_default/ +ipython_config.py + +# pyenv +# For a library or package, you might want to ignore these files since the code is +# intended to run in multiple environments; otherwise, check them in: +# .python-version + +# pipenv +# According to pypa/pipenv#598, it is recommended to include Pipfile.lock in version control. +# However, in case of collaboration, if having platform-specific dependencies or dependencies +# having no cross-platform support, pipenv may install dependencies that don't work, or not +# install all needed dependencies. +#Pipfile.lock + +# PEP 582; used by e.g. github.com/David-OConnor/pyflow +__pypackages__/ + +# Celery stuff +celerybeat-schedule +celerybeat.pid + +# SageMath parsed files +*.sage.py + +# Environments +.env +.venv +env/ +venv/ +ENV/ +env.bak/ +venv.bak/ + +# Spyder project settings +.spyderproject +.spyproject + +# Rope project settings +.ropeproject + +# mkdocs documentation +/site + +# mypy +.mypy_cache/ +.dmypy.json +dmypy.json + +# Pyre type checker +.pyre/ + +# pytype static type analyzer +.pytype/ + +# Cython debug symbols +cython_debug/ + +#special +.desc \ No newline at end of file diff --git a/clients/python/LICENSE b/clients/python/LICENSE new file mode 100644 index 00000000..593b6302 --- /dev/null +++ b/clients/python/LICENSE @@ -0,0 +1,201 @@ + Apache License + Version 2.0, January 2004 + http://www.apache.org/licenses/ + + TERMS AND CONDITIONS FOR USE, REPRODUCTION, AND DISTRIBUTION + + 1. Definitions. + + "License" shall mean the terms and conditions for use, reproduction, + and distribution as defined by Sections 1 through 9 of this document. + + "Licensor" shall mean the copyright owner or entity authorized by + the copyright owner that is granting the License. + + "Legal Entity" shall mean the union of the acting entity and all + other entities that control, are controlled by, or are under common + control with that entity. For the purposes of this definition, + "control" means (i) the power, direct or indirect, to cause the + direction or management of such entity, whether by contract or + otherwise, or (ii) ownership of fifty percent (50%) or more of the + outstanding shares, or (iii) beneficial ownership of such entity. + + "You" (or "Your") shall mean an individual or Legal Entity + exercising permissions granted by this License. + + "Source" form shall mean the preferred form for making modifications, + including but not limited to software source code, documentation + source, and configuration files. + + "Object" form shall mean any form resulting from mechanical + transformation or translation of a Source form, including but + not limited to compiled object code, generated documentation, + and conversions to other media types. + + "Work" shall mean the work of authorship, whether in Source or + Object form, made available under the License, as indicated by a + copyright notice that is included in or attached to the work + (an example is provided in the Appendix below). + + "Derivative Works" shall mean any work, whether in Source or Object + form, that is based on (or derived from) the Work and for which the + editorial revisions, annotations, elaborations, or other modifications + represent, as a whole, an original work of authorship. For the purposes + of this License, Derivative Works shall not include works that remain + separable from, or merely link (or bind by name) to the interfaces of, + the Work and Derivative Works thereof. + + "Contribution" shall mean any work of authorship, including + the original version of the Work and any modifications or additions + to that Work or Derivative Works thereof, that is intentionally + submitted to Licensor for inclusion in the Work by the copyright owner + or by an individual or Legal Entity authorized to submit on behalf of + the copyright owner. For the purposes of this definition, "submitted" + means any form of electronic, verbal, or written communication sent + to the Licensor or its representatives, including but not limited to + communication on electronic mailing lists, source code control systems, + and issue tracking systems that are managed by, or on behalf of, the + Licensor for the purpose of discussing and improving the Work, but + excluding communication that is conspicuously marked or otherwise + designated in writing by the copyright owner as "Not a Contribution." + + "Contributor" shall mean Licensor and any individual or Legal Entity + on behalf of whom a Contribution has been received by Licensor and + subsequently incorporated within the Work. + + 2. Grant of Copyright License. Subject to the terms and conditions of + this License, each Contributor hereby grants to You a perpetual, + worldwide, non-exclusive, no-charge, royalty-free, irrevocable + copyright license to reproduce, prepare Derivative Works of, + publicly display, publicly perform, sublicense, and distribute the + Work and such Derivative Works in Source or Object form. + + 3. Grant of Patent License. Subject to the terms and conditions of + this License, each Contributor hereby grants to You a perpetual, + worldwide, non-exclusive, no-charge, royalty-free, irrevocable + (except as stated in this section) patent license to make, have made, + use, offer to sell, sell, import, and otherwise transfer the Work, + where such license applies only to those patent claims licensable + by such Contributor that are necessarily infringed by their + Contribution(s) alone or by combination of their Contribution(s) + with the Work to which such Contribution(s) was submitted. If You + institute patent litigation against any entity (including a + cross-claim or counterclaim in a lawsuit) alleging that the Work + or a Contribution incorporated within the Work constitutes direct + or contributory patent infringement, then any patent licenses + granted to You under this License for that Work shall terminate + as of the date such litigation is filed. + + 4. Redistribution. You may reproduce and distribute copies of the + Work or Derivative Works thereof in any medium, with or without + modifications, and in Source or Object form, provided that You + meet the following conditions: + + (a) You must give any other recipients of the Work or + Derivative Works a copy of this License; and + + (b) You must cause any modified files to carry prominent notices + stating that You changed the files; and + + (c) You must retain, in the Source form of any Derivative Works + that You distribute, all copyright, patent, trademark, and + attribution notices from the Source form of the Work, + excluding those notices that do not pertain to any part of + the Derivative Works; and + + (d) If the Work includes a "NOTICE" text file as part of its + distribution, then any Derivative Works that You distribute must + include a readable copy of the attribution notices contained + within such NOTICE file, excluding those notices that do not + pertain to any part of the Derivative Works, in at least one + of the following places: within a NOTICE text file distributed + as part of the Derivative Works; within the Source form or + documentation, if provided along with the Derivative Works; or, + within a display generated by the Derivative Works, if and + wherever such third-party notices normally appear. The contents + of the NOTICE file are for informational purposes only and + do not modify the License. You may add Your own attribution + notices within Derivative Works that You distribute, alongside + or as an addendum to the NOTICE text from the Work, provided + that such additional attribution notices cannot be construed + as modifying the License. + + You may add Your own copyright statement to Your modifications and + may provide additional or different license terms and conditions + for use, reproduction, or distribution of Your modifications, or + for any such Derivative Works as a whole, provided Your use, + reproduction, and distribution of the Work otherwise complies with + the conditions stated in this License. + + 5. Submission of Contributions. Unless You explicitly state otherwise, + any Contribution intentionally submitted for inclusion in the Work + by You to the Licensor shall be under the terms and conditions of + this License, without any additional terms or conditions. + Notwithstanding the above, nothing herein shall supersede or modify + the terms of any separate license agreement you may have executed + with Licensor regarding such Contributions. + + 6. Trademarks. This License does not grant permission to use the trade + names, trademarks, service marks, or product names of the Licensor, + except as required for reasonable and customary use in describing the + origin of the Work and reproducing the content of the NOTICE file. + + 7. Disclaimer of Warranty. Unless required by applicable law or + agreed to in writing, Licensor provides the Work (and each + Contributor provides its Contributions) on an "AS IS" BASIS, + WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or + implied, including, without limitation, any warranties or conditions + of TITLE, NON-INFRINGEMENT, MERCHANTABILITY, or FITNESS FOR A + PARTICULAR PURPOSE. You are solely responsible for determining the + appropriateness of using or redistributing the Work and assume any + risks associated with Your exercise of permissions under this License. + + 8. Limitation of Liability. In no event and under no legal theory, + whether in tort (including negligence), contract, or otherwise, + unless required by applicable law (such as deliberate and grossly + negligent acts) or agreed to in writing, shall any Contributor be + liable to You for damages, including any direct, indirect, special, + incidental, or consequential damages of any character arising as a + result of this License or out of the use or inability to use the + Work (including but not limited to damages for loss of goodwill, + work stoppage, computer failure or malfunction, or any and all + other commercial damages or losses), even if such Contributor + has been advised of the possibility of such damages. + + 9. Accepting Warranty or Additional Liability. While redistributing + the Work or Derivative Works thereof, You may choose to offer, + and charge a fee for, acceptance of support, warranty, indemnity, + or other liability obligations and/or rights consistent with this + License. However, in accepting such obligations, You may act only + on Your own behalf and on Your sole responsibility, not on behalf + of any other Contributor, and only if You agree to indemnify, + defend, and hold each Contributor harmless for any liability + incurred by, or claims asserted against, such Contributor by reason + of your accepting any such warranty or additional liability. + + END OF TERMS AND CONDITIONS + + APPENDIX: How to apply the Apache License to your work. + + To apply the Apache License to your work, attach the following + boilerplate notice, with the fields enclosed by brackets "[]" + replaced with your own identifying information. (Don't include + the brackets!) The text should be enclosed in the appropriate + comment syntax for the file format. We also recommend that a + file or class name and description of purpose be included on the + same "printed page" as the copyright notice for easier + identification within third-party archives. + + Copyright [2021] [odpf] + + Licensed under the Apache License, Version 2.0 (the "License"); + you may not use this file except in compliance with the License. + You may obtain a copy of the License at + + http://www.apache.org/licenses/LICENSE-2.0 + + Unless required by applicable law or agreed to in writing, software + distributed under the License is distributed on an "AS IS" BASIS, + WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + See the License for the specific language governing permissions and + limitations under the License. diff --git a/clients/python/README.md b/clients/python/README.md new file mode 100644 index 00000000..47d1aaf7 --- /dev/null +++ b/clients/python/README.md @@ -0,0 +1,69 @@ +# Stencil Python client + +[![PyPI version](https://badge.fury.io/py/stencil-python-client.svg)](https://badge.fury.io/py/stencil-python-client) + +Stencil Python client package provides a store to lookup protobuf descriptors and options to keep the protobuf descriptors upto date. + +It has following features + - Deserialize protobuf messages directly by specifying protobuf message name + - Ability to refresh protobuf descriptors in specified intervals + - Support to download descriptors from multiple urls + + +## Requirements + + - Python 3.7+ + +## Installation + +Use `pip` +``` +pip3 install stencil-python-client +``` + +Then import the stencil package into your own code as mentioned below +```python +from odpf import stencil +``` + +## Usage + +### Creating a client + +```python +from odpf import stencil + +url = "http://url/to/proto/descriptorset/file" +client = stencil.Client(url) +``` + +### Creating a multiURLClient + +```python +from odpf import stencil + +urls = ["http://urlA", "http://urlB"] +client = stencil.MultiUrlClient(urls) +``` + +### Get Descriptor +```python +from odpf import stencil + +url = "http://url/to/proto/descriptorset/file" +client = stencil.Client(url) +client.get_descriptor("google.protobuf.DescriptorProto") +``` + +### Parse protobuf message. +```python +from odpf import stencil + +url = "http://url/to/proto/descriptorset/file" +client = stencil.Client(url) + +data = "" +desc = client.parse("google.protobuf.DescriptorProto", data) +``` + +Refer to [stencil documentation](https://odpf.gitbook.io/stencil/) for more information what you can do in stencil. From ea313e1938c63e3249ce1dea5f2635059eb4b611 Mon Sep 17 00:00:00 2001 From: Muhamad Tohir Date: Sat, 18 Sep 2021 09:37:40 +0700 Subject: [PATCH 08/12] feat: add setup for pypi package --- clients/python/setup.py | 26 ++++++++++++++++++++++++++ 1 file changed, 26 insertions(+) create mode 100644 clients/python/setup.py diff --git a/clients/python/setup.py b/clients/python/setup.py new file mode 100644 index 00000000..b526779d --- /dev/null +++ b/clients/python/setup.py @@ -0,0 +1,26 @@ +import setuptools + +with open("README.md", "r", encoding="utf-8") as fh: + long_description = fh.read() + +setuptools.setup( + name="stencil-python-client", + version="0.0.1", + author="ODPF", + author_email="odpf@gmail.com", + description="Stencil Python client package provides a store to lookup protobuf descriptors and options to keep the protobuf descriptors upto date.", + long_description=long_description, + long_description_content_type="text/markdown", + url="https://github.com/odpf/stencil", + project_urls={ + "Bug Tracker": "https://github.com/odpf/stencil/issues", + }, + classifiers=[ + "Programming Language :: Python :: 3", + "License :: OSI Approved :: Apache Software License", + "Operating System :: OS Independent", + ], + package_dir={"": "src"}, + packages=setuptools.find_packages(where="src"), + python_requires=">=3.6", +) \ No newline at end of file From 22f84298d66a56b72fc07910210b486eb932ae4e Mon Sep 17 00:00:00 2001 From: Muhamad Tohir Date: Sat, 18 Sep 2021 09:40:25 +0700 Subject: [PATCH 09/12] fix: add new line --- clients/python/.gitignore | 3 --- 1 file changed, 3 deletions(-) diff --git a/clients/python/.gitignore b/clients/python/.gitignore index a42d994c..a81c8ee1 100644 --- a/clients/python/.gitignore +++ b/clients/python/.gitignore @@ -136,6 +136,3 @@ dmypy.json # Cython debug symbols cython_debug/ - -#special -.desc \ No newline at end of file From 9bffdceadc618503f10331447cfe7a6d9ec5c115 Mon Sep 17 00:00:00 2001 From: Muhamad Tohir Date: Sat, 18 Sep 2021 09:43:07 +0700 Subject: [PATCH 10/12] fix: add new line at the end of file --- clients/python/.gitignore | 3 +++ clients/python/conftest.py | 1 - clients/python/setup.py | 2 +- clients/python/src/odpf/stencil.py | 2 +- 4 files changed, 5 insertions(+), 3 deletions(-) diff --git a/clients/python/.gitignore b/clients/python/.gitignore index a81c8ee1..ee3b4274 100644 --- a/clients/python/.gitignore +++ b/clients/python/.gitignore @@ -136,3 +136,6 @@ dmypy.json # Cython debug symbols cython_debug/ + +#special +.desc diff --git a/clients/python/conftest.py b/clients/python/conftest.py index b1948e36..02a545de 100644 --- a/clients/python/conftest.py +++ b/clients/python/conftest.py @@ -11,4 +11,3 @@ def protoc_setup(): input_dir = os.path.join(current_dir, 'test/data') run(['protoc',f'--descriptor_set_out={output_file}','--include_imports',f'--proto_path={input_dir}','one.proto'], cwd=current_dir) - \ No newline at end of file diff --git a/clients/python/setup.py b/clients/python/setup.py index b526779d..f6e87660 100644 --- a/clients/python/setup.py +++ b/clients/python/setup.py @@ -23,4 +23,4 @@ package_dir={"": "src"}, packages=setuptools.find_packages(where="src"), python_requires=">=3.6", -) \ No newline at end of file +) diff --git a/clients/python/src/odpf/stencil.py b/clients/python/src/odpf/stencil.py index a3d5d5b3..6109255b 100644 --- a/clients/python/src/odpf/stencil.py +++ b/clients/python/src/odpf/stencil.py @@ -27,4 +27,4 @@ def parse(self, name:str, data:str): class Client(MultiUrlClient): def __init__(self, url: str) -> None: - super().__init__([url]) \ No newline at end of file + super().__init__([url]) From 3e46f768cb640e9d2e69f3289a5e610c2b68a053 Mon Sep 17 00:00:00 2001 From: Muhamad Tohir Date: Sat, 18 Sep 2021 09:52:21 +0700 Subject: [PATCH 11/12] docs: add pypi link --- clients/python/README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/clients/python/README.md b/clients/python/README.md index 47d1aaf7..f64d92e0 100644 --- a/clients/python/README.md +++ b/clients/python/README.md @@ -1,6 +1,6 @@ # Stencil Python client -[![PyPI version](https://badge.fury.io/py/stencil-python-client.svg)](https://badge.fury.io/py/stencil-python-client) +[![PyPI version](https://badge.fury.io/py/stencil-python-client.svg)](https://pypi.org/project/stencil-python-client) Stencil Python client package provides a store to lookup protobuf descriptors and options to keep the protobuf descriptors upto date. From 1b436d581618ce13dc9a375ab3971a65c5507e40 Mon Sep 17 00:00:00 2001 From: Muhamad Tohir Date: Mon, 20 Sep 2021 12:22:02 +0700 Subject: [PATCH 12/12] fix: change deserialization data to bytes --- clients/python/src/odpf/stencil.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/clients/python/src/odpf/stencil.py b/clients/python/src/odpf/stencil.py index 6109255b..6fc9228e 100644 --- a/clients/python/src/odpf/stencil.py +++ b/clients/python/src/odpf/stencil.py @@ -21,7 +21,7 @@ def refresh(self): def get_descriptor(self, name:str) -> Message: return self._store.get(name) - def parse(self, name:str, data:str): + def parse(self, name:str, data:bytes): msg = self.get_descriptor(name) return msg.ParseFromString(data)