[jira] [Commented] (CASSANDRA-17271) Remove unused dependency of geomet from cqlsh.py

[Brandon Williams (Jira)]

    [ 
https://issues.apache.org/jira/browse/CASSANDRA-17271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17478995#comment-17478995
 ] 

Brandon Williams commented on CASSANDRA-17271:
----------------------------------------------

I see the PR is against trunk but it applies cleanly to 4.0 and seems fine to 
do there too:

||Branch||CI||
|[4.0|https://github.com/driftx/cassandra/tree/CASSANDRA-17271]|[circle|https://app.circleci.com/pipelines/github/driftx/cassandra?branch=CASSANDRA-17271&filter=all]|
|[trunk|https://github.com/driftx/cassandra/tree/CASSANDRA-17271-trunk]|[circle|https://app.circleci.com/pipelines/github/driftx/cassandra?branch=CASSANDRA-17271-trunk&filter=all]|

> Remove unused dependency of geomet from cqlsh.py
> ------------------------------------------------
>
>                 Key: CASSANDRA-17271
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-17271
>             Project: Cassandra
>          Issue Type: Task
>          Components: CQL/Interpreter
>            Reporter: Brad Schoening
>            Assignee: Brad Schoening
>            Priority: Normal
>             Fix For: 4.0.x, 4.x
>
>
> The python library *geomet* is used (optionally) by a unit test in the python 
> driver, but was made optional in v3.24.0 of the cassandra-driver:
>  * 👉 Make geomet an optional dependency at runtime (PYTHON-1237)
> It's inclusion on the sys.path for cqlsh would appear unnecessary and unused.
>  
> cqlsh.py:
> {quote}third_parties = ('futures-', 'six-', 'geomet-'){quote}
> {quote}for lib in third_parties:
>     lib_zip = find_zip(lib)
>     if lib_zip:
>         sys.path.insert(0, lib_zip){quote}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org