skorper commented on pull request #132:
URL: 
https://github.com/apache/incubator-sdap-nexus/pull/132#issuecomment-930323278


   Further updated the backwards compatibility of the solr doc -- works when 
the `tile_standard_name_s` field is not present. Tested this on dataset 
`ascat-l2-coastal-multi-t1`. Sample solr doc:
   
   ```json
   {
           "table_s":"sea_surface_temp",
           "geo":"POLYGON((29.305 40.166, 43.460 40.166, 43.460 45.753, 29.305 
45.753, 29.305 40.166))",
           "id":"7b2b099e-5a21-3b92-8d89-ad2235db4389",
           
"solr_id_s":"ascat-l2-coastal-multi-t1!7b2b099e-5a21-3b92-8d89-ad2235db4389",
           "sectionSpec_s":"NUMCELLS:30:60,NUMROWS:360:390",
           "dataset_s":"ascat-l2-coastal-multi-t1",
           
"granule_s":"ascat_20170102_182400_metopb_22278_eps_o_coa_2401_ovw.l2.nc",
           "tile_var_name_s":"[\"wind_speed\", \"wind_to_direction\"]",
           "tile_min_lon":[29.30514],
           "tile_max_lon":[43.45994],
           "tile_min_lat":[40.16612000000001],
           "tile_max_lat":[45.752610000000004],
           "tile_depth":[0.0],
           "tile_min_time_dt":"2017-01-02T18:35:15Z",
           "tile_max_time_dt":"2017-01-02T18:36:09Z",
           "tile_min_val_d":1.3299999237060547,
           "tile_max_val_d":252.8000030517578,
           "tile_avg_val_d":33.11838150024414,
           "tile_count_i":526,
           "_version_":1708470605235355648
   }
   ```
   
   Confirmed matchup works as expected with this solr doc. Tested the following 
cases with this latest change:
   
   1. Solr doc with `tile_var_name_s` but no `tile_standard_name_s`
   2. Solr doc with `tile_var_name_s` and `tile_standard_name_s`
   3. Solr doc with `tile_var_name_ss`
   
   All work as expected.


-- 
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: dev-unsubscr...@sdap.apache.org

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


Reply via email to