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/09/21 02:10:08 UTC

[GitHub] [incubator-sdap-ingester] skorper opened a new pull request #41: SDAP-323: Update summarizing processor and Solr schema to support multiple variables

skorper opened a new pull request #41:
URL: https://github.com/apache/incubator-sdap-ingester/pull/41


   Updated the solr doc to support multiple variables. The following changes were made:
   
   1. Updated var name field to a list type
   2. Updated var name field to `tile_var_name_ss`
   3. Added a new field `{var_name}.tile_standard_name_s` which contains standard name.
   
   For example:
   
   ```json
   ...
     "tile_var_name_ss": [
       "wind_speed",
       "wind_to_direction"
     ],
     "wind_speed.tile_standard_name_s": "wind_speed",
   
     "wind_to_direction.tile_standard_name_s": "wind_to_direction",
   ...
   ```
   
   The variable name and standard name are still stored as json encoded lists OR strings (single vs multi-var), but then are translated to lists in the solr metadata. In the single-var case, the var name field is a list of size 1, in the multi-var case the var name is a list of size N. 
   
   I implemented it such that standard name is `null` in the solr doc when not available in the granule metadata. Any thoughts about whether or not this is the desired behavior?


-- 
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-unsubscribe@sdap.apache.org

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

[GitHub] [incubator-sdap-ingester] skorper commented on pull request #41: SDAP-323: Update summarizing processor and Solr schema to support multiple variables

Posted by GitBox <gi...@apache.org>.
skorper commented on pull request #41:
URL: https://github.com/apache/incubator-sdap-ingester/pull/41#issuecomment-925394198


   Don't index null field.


-- 
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-unsubscribe@sdap.apache.org

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

[GitHub] [incubator-sdap-ingester] skorper commented on pull request #41: SDAP-323: Update summarizing processor and Solr schema to support multiple variables

Posted by GitBox <gi...@apache.org>.
skorper commented on pull request #41:
URL: https://github.com/apache/incubator-sdap-ingester/pull/41#issuecomment-925410165


   @ngachung I've updated the ingester so the standard_name field is only included if that value is non-null. 


-- 
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-unsubscribe@sdap.apache.org

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

[GitHub] [incubator-sdap-ingester] ngachung merged pull request #41: SDAP-323: Update summarizing processor and Solr schema to support multiple variables

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


   


-- 
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-unsubscribe@sdap.apache.org

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