search

aws / amazon-finspace-examples

This repo contains sample code and sample notebooks to illustrate how to work with Amazon FinSpace
MIT No Attribution
20 stars 25 forks source link

Fix .aws.list_kx_clusters[] and typo in comment #17

Closed peterlarson closed 12 months ago

peterlarson commented 12 months ago

Today .aws.list_kx_clusters will fail if all the clusters returned do not have a description. With this fix the return will still work.

Current behavior: 

q).aws.list_kx_clusters[]
'clusterDescription
  [1]  /Users/pelarso/code/fork-amazon-finspace-examples/ManagedkdbInsights/q/aws.q:61: .aws.list_kx_clusters:
   clusters: list_kx_clusters_full[];
   select cluster_name: clusterName, status, cluster_type:clusterType, description:clusterDescription from clusters
   ^
 }

With this change: 

q).aws.list_kx_clusters[]
cluster_name            status    cluster_type description
----------------------------------------------------------
"DEVCluster-regression" "RUNNING" "DEV"        ""
"RDBCluster-regression" "RUNNING" "RDB"        ""
"HDBCluster-regression" "RUNNING" "HDB"        ""

This PR also includes a fix to the documentation of .aws.update_kx_cluster_databases

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.