dsmiley opened a new pull request, #3171:
URL: https://github.com/apache/solr/pull/3171

   See this 
[thread](https://github.com/apache/solr/pull/254#discussion_r1917413526)
   > I see a repeating failure running DockMakerTest.testBasicCardinalityAlpha 
with seed E0BBCB76FA600308 in which every field value is the letter 'k'. This 
DSL framework is extremely different from any other code most people encounter 
so it took some time to narrow down the problem to where I'm commenting. This 
code will choose consecutive seeds for subsequent processing. (Consecutiveness 
is not a problem I think). For a cardinality of 2 and based on the random test 
seed, this long seed will be either 635258717 or 635258718. But the next stage 
will always produce 'k' for either of those seeds; the next stage produces a 
random string of length from 1 to 6.
   
   I think the fundamental fix is in the test; to recognize this is a 
*max*Cardinality, and weaken its assertions accordingly.
   
   I added some comments in StringsDSL.


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org

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


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

Reply via email to