epugh commented on code in PR #484:
URL: https://github.com/apache/solr/pull/484#discussion_r1030846318


##########
solr/server/solr/configsets/sample_techproducts_configs/conf/managed-schema:
##########
@@ -195,39 +195,35 @@
 
 
    <!-- Dynamic field definitions allow using convention over configuration
-       for fields via the specification of patterns to match field names.
-       EXAMPLE:  name="*_i" will match any field ending in _i (like myid_i, 
z_i)
-       RESTRICTION: the glob-like pattern in the name attribute must have
-       a "*" only at the start or the end.  -->
-
-   <dynamicField name="*_i"  type="pint"    indexed="true"  stored="true"/>
-   <dynamicField name="*_is" type="pint"    indexed="true"  stored="true"  
multiValued="true"/>
-   <dynamicField name="*_s"  type="string"  indexed="true"  stored="true" />
-   <dynamicField name="*_s_ns"  type="string"  indexed="true"  stored="false" 
/>
-   <dynamicField name="*_ss" type="string"  indexed="true"  stored="true" 
multiValued="true"/>
-   <dynamicField name="*_l"  type="plong"   indexed="true"  stored="true"/>
-   <dynamicField name="*_l_ns"  type="plong"   indexed="true"  stored="false"/>
-   <dynamicField name="*_ls" type="plong"   indexed="true"  stored="true"  
multiValued="true"/>
-   <dynamicField name="*_t"  type="text_general"    indexed="true"  
stored="true"/>
-   <dynamicField name="*_txt" type="text_general"   indexed="true"  
stored="true" multiValued="true"/>
-   <dynamicField name="*_en"  type="text_en"    indexed="true"  stored="true" 
multiValued="true"/>
-   <dynamicField name="*_b"  type="boolean" indexed="true" stored="true"/>
-   <dynamicField name="*_bs" type="boolean" indexed="true" stored="true"  
multiValued="true"/>
-   <dynamicField name="*_f"  type="pfloat"  indexed="true"  stored="true"/>
-   <dynamicField name="*_fs" type="pfloat"  indexed="true"  stored="true"  
multiValued="true"/>
-   <dynamicField name="*_d"  type="pdouble" indexed="true"  stored="true"/>
-   <dynamicField name="*_ds" type="pdouble" indexed="true"  stored="true"  
multiValued="true"/>
-
+      for fields via the specification of patterns to match field names.
+      EXAMPLE:  name="*_i" will match any field ending in _i (like myid_i, z_i)
+      RESTRICTION: the glob-like pattern in the name attribute must have a "*" 
only at the start or the end.  -->
+
+   <dynamicField name="*_i"   type="pint"     indexed="true"  stored="true"/>
+   <dynamicField name="*_is"  type="pints"    indexed="true"  stored="true"/>
+   <dynamicField name="*_s"   type="string"   indexed="true"  stored="true" />
+   <dynamicField name="*_ss"  type="strings"  indexed="true"  stored="true"/>
+   <dynamicField name="*_l"   type="plong"    indexed="true"  stored="true"/>
+   <dynamicField name="*_ls"  type="plongs"   indexed="true"  stored="true"/>
+   <dynamicField name="*_b"   type="boolean"  indexed="true"  stored="true"/>
+   <dynamicField name="*_bs"  type="booleans" indexed="true"  stored="true"/>
+   <dynamicField name="*_f"   type="pfloat"   indexed="true"  stored="true"/>
+   <dynamicField name="*_fs"  type="pfloats"  indexed="true"  stored="true"/>
+   <dynamicField name="*_d"   type="pdouble"  indexed="true"  stored="true"/>
+   <dynamicField name="*_ds"  type="pdoubles" indexed="true"  stored="true"/>
    <dynamicField name="*_dt"  type="pdate"    indexed="true"  stored="true"/>
-   <dynamicField name="*_dts" type="pdate"    indexed="true"  stored="true" 
multiValued="true"/>
-   <dynamicField name="*_p"  type="location" indexed="true" stored="true"/>
+   <dynamicField name="*_dts" type="pdates"   indexed="true"  stored="true"/>
+   <dynamicField name="*_t"   type="text_general" indexed="true" stored="true" 
multiValued="false"/>
+   <dynamicField name="*_txt" type="text_general" indexed="true" 
stored="true"/>
 
-   <dynamicField name="*_c"   type="currency" indexed="true"  stored="true"/>
+   <dynamicField name="random_*" type="random"/>
+   <dynamicField name="ignored_*" type="ignored"/>
 
-   <dynamicField name="ignored_*" type="ignored" multiValued="true"/>
-   <dynamicField name="attr_*" type="text_general" indexed="true" 
stored="true" multiValued="true"/>

Review Comment:
   thanks @thomascorthals for asking!     I want to move us in the direction of 
having a couple of configsets that we ship with Solr that are actually meant to 
be used by end users...   Like `_default`.   Right now, 
`sample_techproducts_configs` is a bit of an odd duck, we use it to demo new 
features in Solr, we use it in our unit tests, and of course, a lot of folks 
base their first search schema on it!   So it's always changing.
   
   I'm also interested in having some configsets that are used by the schema 
designer that you can import and then use in your own schemas...   There is a 
PR out there for that...
   
   However, back to your original question, I would recommend `_default` for 
backwards compat testing.   Or maybe we want to have a new saner/simpler 
`basic` schema?   Or `ecommerce` schema?   I also wonder if we should think 
about this in light of some of the work to have benchmarks....



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