You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sdap.apache.org by GitBox <gi...@apache.org> on 2021/05/18 22:39:14 UTC

[GitHub] [incubator-sdap-nexus] ngachung opened a new pull request #128: SDAP-308 Fixed deletebyquery.py to be compatible with Python 3

ngachung opened a new pull request #128:
URL: https://github.com/apache/incubator-sdap-nexus/pull/128


   Updated solrcloudpy dependency and removed eval so that deletebyquery tool works with Python 3.
   
   Tested this locally and confirmed input works.
   
   Before the change:
   
   ```
   Query found 100254 matching documents. Continue? [y]/n/(s)ample: n
   Traceback (most recent call last):
     File "/Users/nchung/PycharmProjects/incubator-sdap-nexus/tools/deletebyquery/deletebyquery.py", line 269, in <module>
       delete_by_query(the_args)
     File "/Users/nchung/PycharmProjects/incubator-sdap-nexus/tools/deletebyquery/deletebyquery.py", line 92, in delete_by_query
       if check_query(query):
     File "/Users/nchung/PycharmProjects/incubator-sdap-nexus/tools/deletebyquery/deletebyquery.py", line 127, in check_query
       do_continue = eval(input("Query found %s matching documents. Continue? [y]/n/(s)ample: " % num_found))
     File "<string>", line 1, in <module>
   NameError: name 'n' is not defined
   ```
   
   After the change:
   
   ```
   Query found 100254 matching documents. Continue? [y]/n/(s)ample: n
   2021-05-18T15:37:34 INFO:root:  Exiting
   ```
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #128: SDAP-308 Fixed deletebyquery.py to be compatible with Python 3

Posted by GitBox <gi...@apache.org>.
ngachung merged pull request #128:
URL: https://github.com/apache/incubator-sdap-nexus/pull/128


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org