Yea, we all forget to close them now and then, although I have started to merge with the PR button, and then it happens automatically :)
Perhaps it is time for our project to write a few «bots» to handle things like this. I’m sure INFRA has a place we can host it. The first bot would be the Lucene-issues-bot which could run once per day and: * Detect PRs whose JIRA is closed, add a label «autoclosed» to it and then close it * Detect new PRs filed without a JIRA link and add a comment about it * Detect open PRs/JIRAs from first-time contributors that have not received committer attention first 7days and send a Slack message about it * Detect open PRs/JIRAs older than 30d without any activity last 7days and add a «nudge» comment to it Later one could do much more, to help tame the loooong backlog. Jan > 15. feb. 2020 kl. 17:03 skrev Ishan Chattopadhyaya > <ichattopadhy...@gmail.com>: > > Thanks Jan. Is there a way to bulk close all of them? If we automate it and > run it weekly, it would be nice. > > On Sat, 15 Feb, 2020, 6:53 AM Erick Erickson, <erickerick...@gmail.com > <mailto:erickerick...@gmail.com>> wrote: > Thanks, closed mine... > > > On Feb 14, 2020, at 6:28 PM, Jan Høydahl <jan....@cominvent.com > > <mailto:jan....@cominvent.com>> wrote: > > > > I made a report with dev-tools/scripts/githubPRs.py which shows «orphan» > > PRs and PRs that have not been closed even if the linked JIRA is closed. > > Please help close the gap. > > You can typically link a PR with a JIRA just by editing the PR title and > > also mentioning the JIRA ID as a comment on the PR. > > > > Lucene/Solr Github PR report > > > > Number of open Pull Requests: 263 > > > > PRs lacking JIRA reference in title (51) > > > > • #1234: Add compression for Binary doc value fields (markharwood) > > • #1143: HdfsDirectory support createTempOutput (kaynewu) > > • #1109: More pervasive use of PackageLoader / PluginInfo (dsmiley) > > • #1090: Update README.txt for analysis-extras (tuftii) > > • #1026: Buffer one record from each shard every 5000 reads to ensure > > connecti… (joel-bernstein) > > • #1005: LUCENE 9042: Refactor TopGroups.merge tests (diegoceccarelli) > > • #993: maybe can simpler (kkewwei) > > • #988: Update README.md (hemantkadyan) > > • #908: Change the file format of README files from README.txt to > > README.md a… (pinkeshsharma) > > • #880: Tweak header format. (markrmiller) > > • #873: Rename README.txt to README.md (matejsoroka) > > • #854: Shared PQ Based Early Termination for Concurrent Search > > (atris) > > • #807: Remove solr.jetty.https.port when SSL is not used > > (upendrasoft) > > • #772: Payload-stored position length intervals (romseygeek) > > • #673: Replace instances of Math.random with Random.nextDouble > > (bd2019us) > > • #669: Facet2d (joel-bernstein) > > • #639: Solve the problem of highlighting Chinese inaccurately. > > (mengbin16) > > • #615: Intervals Query Parser (jmorph99) > > • #602: docu change: use class TopDocs instead of Hits (saschaszott) > > • #601: Adding reader settings for moving fst offheap (jainankitk) > > • #596: Under this branch, the dataDimensionCount is definitely not > > zero. (hanbj) > > • #542: LuceneLevenshteinDistance computes distance values outside of > > interval [0, 1] (saschaszott) > > • #508: Simplified JAVA_VER_NUM to utilize single expr execution > > (LinkMJB) > > • #492: Answer to TODO: Replace Manual Encoding with JSON Module > > (MarcusSorealheis) > > • #491: Update for multiple term's suggestion scoring (UtsavVanodiya7) > > • #484: solr 7.5 suggest The recommended result is empty (lunxian8) > > • #478: Query Source Tracker custom component (tirthmehta1994) > > • #450: Add rule exception for "imento" and "mento" suffix > > (ceciliassis) > > • #442: Fixing an edge case bug when overriding a default > > PostingsSolrHighligher (Apmats) > > • #405: don't die when java prints tool options (tjakway) > > • #404: Comment to explain how to use URLClassifyProcessorFactory > > (ohtwadi) > > • #399: fix explicit type declaration (nvnmandadhi) > > • #383: In ContextQuery, use a more comprehensive check for an empty > > prefix automaton. (jtibshirani) > > • #350: SOLR match mode change for the rouding off instead of taking > > floor (MighTguY) > > • #340: Add SolrConfig to SolrRequestParsers constructor in > > EmbeddedSolrServer (squallsama) > > • #309: Update ZkConfigManager.java (yshhuang) > > • #308: Add a suggester that operates on tokenized values from a > > field (cbeer) > > • #293: spellcheck prefix already contains a trailing dot > > (kevinlacire) > > • #241: SpanishLightStemmer fix for plural words like casas > > (JONAF2103) > > • #231: WordDelimiterGraphFilter: Better support for camel case > > splitting. (niqueco) > > • #218: feat: Separate SuggestModes for WordBreak and WordJoin > > (abhidemon) > > • #201: Allocate ArrayList with exact size (kno10) > > • #175: Skipping merger (damianparus) > > • #153: Fix issues reported by findbugs (praste) > > • #152: Added log prior calculations in CachingNaiveBayesClassifier. > > (KevinLeeCrosby) > > • #131: Fix peer sync replcation test check (praste) > > • #124: fix small issue in solr shell script (idler) > > • #110: Update SearchFiles.java (FreekDB) > > • #85: Allow updating configs like port # on forced upgrade > > (jasondemorrow) > > • #48: moved common string to constant file (vcharmcaster) > > • #8: Do not log error messages, if client has been interrupted > > (markus-s24) > > Open PRs with a resolved JIRA (22) > > > > • PR #1036: Resolved 2019-11-25 SOLR-13967: Update query.css to make > > query form sticky on the top while scrolling page (None) > > • PR #1022: Closed 2019-12-11 SOLR-13953: Update eviction behavior of > > cache in Solr Prometheus exporter to allow for larger clusters > > (erickerickson) > > • PR #942: Resolved 2019-10-15 SOLR-13834: > > ZkController#getSolrCloudManager() now uses the same ZkStateReader > > (ichattopadhyaya) > > • PR #932: Resolved 2019-10-11 SOLR-13829: Stop converting continuous > > numbers to Longs in Streaming Expressions (None) > > • PR #1048: Closed 2019-12-10 SOLR-13806: change type of > > QueryResponse._explainMap (ab) > > • PR #922: Resolved 2019-10-15 SOLR-13793: limit number of remote > > queries to number of shard replicas (ichattopadhyaya) > > • PR #976: Resolved 2019-12-18 SOLR-13749: Implement support for > > joining across collections with multiple shards (gus) > > • PR #1260: Closed 2019-11-30 SOLR-13669: DIH: Add System property > > toggle for use of dataConfig param (dsmiley) > > • PR #812: Resolved 2019-08-14 [SOLR-13663] Span Positional Range > > Support in XML Query Parser + test (mkhl) > > • PR #1016: Closed 2019-12-12 SOLR-13662: Test fix & Reference guide > > for package manager (ichattopadhyaya) > > • PR #915: Closed 2019-12-12 SOLR-13661: Package management APIs, > > Package loading, Package store (ichattopadhyaya) > > • PR #829: Resolved 2019-12-02 SOLR-13452: Update the lucene-solr > > build from Ivy+Ant+Maven (shadow build) to Gradle. (dweiss) > > • PR #1122: Resolved 2020-01-15 SOLR-12490 Added parsing of > > json.queries for referring in JSON facets (mkhl) > > • PR #365: Closed 2018-11-05 [SOLR-12243] span query generalization + > > query parser tests (sarowe) > > • PR #1064: Resolved 2020-01-09 LUCENE-9084: circular synchronization > > wait (potential deadlock) in AnalyzingInfixSuggester (mikemccand) > > • PR #906: Closed 2019-12-09 LUCENE-8996: maxScore is sometimes > > missing from distributed responses (cpoerschke) > > • PR #631: Resolved 2019-04-03 LUCENE-8750: implement setMissingValue > > for ValueSource sortFields (romseygeek) > > • PR #536: Resolved 2019-01-17 LUCENE-8643: Decrease test complexity > > in the default case. Exclude simple text codec. (dweiss) > > • PR #533: Resolved 2019-01-15 LUCENE-8636: TestPointQueries and long > > execution times (dweiss) > > • PR #529: Resolved 2019-12-18 LUCENE-8617: Use SimpleFSDirectory on > > non-default FS (None) > > • PR #500: Resolved 2019-12-18 LUCENE-8517: do not wrap > > FixedShingleFilter with conditional in TestR… (None) > > • PR #543: Resolved 2019-01-28 LUCENE-8474: final cleanups and > > removal of RAMDirectory (dweiss) > > > > > > This report is generated as follows: > > > > dev-tools/scripts/githubPRs.py —html >report.html && open report.html > > > > …then copy/pasted into the email :) > > > > Jan > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org > <mailto:dev-unsubscr...@lucene.apache.org> > For additional commands, e-mail: dev-h...@lucene.apache.org > <mailto:dev-h...@lucene.apache.org> >
