First off, I think we need to figure out which version of "phoenixdb" we want to include. Technically, I don't know which would be preferred: DB API 2.0 or SQLAlchemy. Do we have to choose just one? I can only say that I know of users have been using Lukas' variant but I don't know if that's because it was the one that existed.

I think Lukas is being very calm/reserved in the fact that his code was re-licensed, his copyright statement removed, and a new copyright statement added. Shame on you, Dimitri -- you should fix this.

I'm not sure how best to integrate this into our Maven project either. I know that there are ways to integrate Python code/tests into our Maven project, but I'm not sure how best to make this work. I also don't know what kind of centralized distribution platforms exist here at the ASF for us to use (is there something similar to Maven's repo mirroring for Python libs?). I see Thrift[1] has some code deployed, but I'm not sure how to do that.

Finally, while I would (selfishly) like this to exist in Avatica, I'm not too selfish to say that it shouldn't exist in Phoenix. The type-specific details that Lukas pointed out are a good reason to keep this in Phoenix for now. Perhaps we can figure out someway later on to pull it in Avatica and just leave something down here in Phoenix that has the necessary pieces overridden.

- Josh

[1] https://pypi.python.org/pypi/thrift/0.9.1

Lukáš Lalinský wrote:
Hi James,

How would you prefer the integration into the Phoenix project to be
handled? I'm more than happy to do it, but I don't see a clear path.

Licensing is not an issue, since it's using the same license as
Phoenix itself. I guess I'd just need to sign the CLA?

Adding it to the Phoenix source repo is just a matter of deciding on
the structure, since there is currently no non-Java code as far as I
know. Any preferences there?

There is a question on whether it should be added to Phoenix or
Avatica. I think Phoenix is a better home for the library, since
everything in there is only tested on Phoenix and not other Avatica
backends and the type handling is Phoenix-specific as well.

Releasing is a different matter. I'm not sure how Phoenix releases are
currently handled, but for a Python library, it makes sense to upload
the package to PyPI since that's from where most Python developers are
getting packages from. Who would do that? If me, I wouldn't want to
block Phoenix releases due to me unavailability, but on the other hand
it would be bad for the releases to get out of sync.

I'm also happy to abandon the project if you integrate Dimitri's version.

Regards,

Lukas



On Fri, Mar 24, 2017 at 1:24 AM, James Taylor<jamestay...@apache.org>  wrote:
Thanks, Dimitri! If you're interested, I think the community would welcome
this as a contribution to Apache Phoenix so it can appear in our regular
distributions.

Regards,
James

On Thu, Mar 23, 2017 at 5:21 PM, Dimitri<gryte...@gmail.com>  wrote:
Hi,

I make some update on my fork of Lukáš Lalinský phoenix python lib.

I integrate sqlalchemy and correct some bug.

you can find the code source on https://github.com/Pirionfr/pyPhoenix
and it's available in https://pypi.python.org/pypi/pyPhoenix

regards
Dimitri Capitaine







2016-11-07 18:12 GMT+01:00 Josh Elser<els...@apache.org>:

+1

I was poking around with it this weekend. I had some issues (trying to
use
it from the Avatica side, instead of PQS, specifically), but for the
most
part it worked. Definitely feel free to report any issues you run into:
https://bitbucket.org/lalinsky/python-phoenixdb. It would be nice to
make
this a "first-class citizen".

James Taylor wrote:

Thanks, Dimitri. I created PHOENIX-3459 to update the website with the
new
link. It'd be great to see this make it into Phoenix proper (or Avatica
or
a combination of the two).

      James

On Sun, Nov 6, 2016 at 12:35 PM, Dimitri<gryte...@gmail.com>   wrote:

erratum
Someone else did the port to protobuf, Lukáš Lalinský merged the pull
request and it's now released as version 0.5.

Dimitri

2016-11-03 22:14 GMT+01:00 Dimitri<gryte...@gmail.com>:

Hi,
Lukáš Lalinský is no longer use Phoenix, so he is not actively
working
on
his lib anymore.

One major problem with his lib is that it uses the JSON interface,
which
is now deprecated.

I fork it for my use (phoenix 4.8 and sqlalchemy.), it's now use
protobuf,I push my code on my github. (https://github.com/Pirionfr/
python-phoenixdb)

For some who want to use it can you put the link on your site (
https://phoenix.apache.org/phoenix_python.html) ?





Reply via email to