Summary: 81 instances, 72 unique Text Count // TODO - can add support for a separate property for key password 1 // TODO #34 - Investigate locale 1 // TODO - implement 3 // TODO - implement through a HEAD or a GET to "/", or a dummy SQL? 1 // TODO: investigate other injection prevention 1 // TODO #36 - Add return code - see above function comment 1 // TODO #36 - Send query to database to get encoding 1 // TODO - when server plugin supports PreparedStatement fully, implement this as a preparedStatment with params 1 *
TODO: Support for authN and authZ with AWS Sigv4 or security plugin. 1 // TODO - once sql plugin supports PreparedStatement fully, do this through a preparedStatement with params 1 # TODO: remove this after #311 in sql repo is fixed 1 // TODO: Remove this special case after fixing all failed UTs 1 # TODO: cli helpers will have to switch to ptk.Style 1 * TODO: https://github.com/opendistro-for-elasticsearch/sql/issues/710 2 // TODO: return type information is disconnected from the function definitions in SQLFunctions. 1 * engine. TODO: geo, ip etc. 1 # TODO add validation for endpoint to avoid the cost of connecting to some obviously invalid endpoint 1 // TODO #35 - Check if they expect NULL or "" when there is no error. 1 data.put("NULLABLE", 2); // TODO Defaulting to 2, need to find a way to check this 1 * TODO: Push down operations to index scan operator as much as possible in future. 1 CONCAT(), // TODO: varargs support required 1 // TODO: Investigate how to find the type of expression (string or number) 1 // TODO: Merge mappings if they are compatible, for text and text/keyword to text/keyword. 1 // TODO #36 - Send query to database to set encoding 1 // TODO #325 (SQL Plugin)- Fix patterns and escape characters for this 1 * TODO: for now just delegate entire logical plan to storage engine. 1 // TODO - implement? 3 // TODO - can apply settings retry & backoff 1 // TODO #630 - Revisit logic of adding tuples for SQLTables & SQLColumns 1 // TODO: Explore other ways to correlate different fields/functions in the query (params?) 1 // //TODO: support all orders and for each OPEAR find his related OPEAR (< is > , EQ is EQ ,etc..) 1 data.put("IS_GENERATEDCOLUMN", ""); // TODO Defaulting to unknown, need to check 1 // TODO - add support? 3 // TODO #324 (SQL Plugin)- Fix patterns and escape characters for this 1 // TODO - support String representations containing Timestamp With Timezone 1 # TODO: consider add evaluator/handler to filter obviously-invalid input, 1 // TODO - add ByteArrayType support 1 * TODO: need to extract an interface for both SQL and PPL action handler and move these 1 * TODO: Refactor as suggested https://github.com/opendistro-for-elasticsearch/sql-jdbc/pull/76#discussion_r421571383 1 // TODO Defaulting to unknown, need to check 1 // TODO #34 - Investigate locale handling on Mac 1 // TODO - limit the amount read from response.getEntity() ? 1 # TODO: add history, refer to pgcli approach 1 // TODO #324 (SQL Plugin)- Should these be unique? 1 // TODO - JDBCType.VARBINARY - byte[] -> Try OpenSearch data type 1 // TODO (#590): Add implementations for remaining ODBC 2.x function 2 // TODO - can add alias selection strategy 1 * TODO: Assume indexName doesn't have wildcard. 1 // TODO tabs are occurring in multiple of 4 (one option is Guava's Strings.padEnd() method) 1 // TODO String separator is being kept to "|" for the time being as using "\t" will require formatting since 1 # TODO: if decided to add row_limit. Refer to pgcli -> main -> line 866. 1 // TODO #324 (SQL Plugin)- evaluate catalog & schema support 2 // TODO: seems like a bug, either fix, or just get rid of for loop and remove the first key 1 // TODO: Is there a legit case when we want to add field into sort for aggregation queries? 1 // Action button with list of tabs, TODO: disable tabArrowRight and tabArrowLeft when no more scrolling is possible 1 // TODO: get error message from CheckConnectionOptions 1 // TODO (#612): Verify wide character conversion 1 * TODO: Scroll doesn't work for aggregation. Support aggregation later. 1 # TODO: Optimize index suggestion to serve indices options only at the needed position, such as 'from' 1 // TODO: Consider removing that condition 1 // TODO - add/check support 1 // TODO #36 - When we fix encoding, we should look into returning a code here. 1 // TODO #324 (SQL Plugin)- Update if OpenSearch extends support for multiple 1 if (ctx.selectElements().star != null) { //TODO: project operator should be required? 1 // TODO (#585): Add option to log to stderr stream 1 //TODO: calculate dynamically on each call 1 // TODO #35 - Go through and add error messages on exit conditions 1 // TODO - implement this? 1 data.put("IS_NULLABLE", ""); // TODO Defaulting to unknown, need to check this 1 // TODO - support percent encoding for URL reserved characters 1 // TODO Defaulting to unknown, need to check this 1 // TODO support java.time.* and JDBCType.TIME_WITH_TIMEZONE/JDBCType.TIMESTAMP_WITH_TIMEZONE 1