RDFLib / rdflib-sqlalchemy

RDFLib store using SQLAlchemy dbapi as back-end
Other
148 stars 34 forks source link

Is "klass" a typo? #61

Closed sharpaper closed 4 years ago

sharpaper commented 4 years ago

Should klass be class instead?

https://github.com/RDFLib/rdflib-sqlalchemy/blob/abe20884d2450106ce89805c8c837d670be1622c/rdflib_sqlalchemy/tables.py#L76

mwatts15 commented 4 years ago

Hi, @sharpaper. No, the use of klass is intentional. You will see it throughout the code. Since columns can be accessed as attributes (see base.py) and class is a reserved word, the convention of klass was chosen to avoid that conflict.