Skip to content

Conversation

@snappy-sachin
Copy link

What changes were proposed in this pull request?

Problem: The default limit for number of streaming query to be displayed is 20. But in the current UI, only 10 queries are displayed. The rest of the queries are displayed when we click on the sort button.

Changes:
 - Data Table config parameter name is changed from "iDisplayLength" to "pageLength".
 - Streaming Queries navigation list, sources and sink tables config parameter "pageLength" is
set to display all entries in it.  
 - Chart title changed from "Aggregation States" to "Aggregation State"

How was this patch tested?

  • Tested manually and by running streaming program with KAFKA.

Please review http://spark.apache.org/contributing.html before opening a pull request.

 - Data Table config parameter name is changed from "iDisplayLength" to "pageLength".
 - Streaming Queries navigation list, sources and sink tables config parameter "pageLength" is
   set to display all entries in it.  
 - Chart title changed from "Aggregation States" to "Aggregation State"
@snappy-sachin
Copy link
Author

Review not needed as these are just configuration parameter changes. Hence merging.

@snappy-sachin snappy-sachin merged commit 745f305 into snappy/branch-2.1 Dec 13, 2019
@snappy-sachin snappy-sachin deleted the SNAP-3273 branch December 13, 2019 15:39
sumwale pushed a commit to sumwale/spark that referenced this pull request Nov 5, 2020
…TIBCOSoftware#196)

 - Data Table config parameter name is changed from "iDisplayLength" to "pageLength".
 - Streaming Queries navigation list, sources and sink tables config parameter "pageLength" is
   set to display all entries in it.  
 - Chart title changed from "Aggregation States" to "Aggregation State"
sumwale pushed a commit that referenced this pull request Jul 11, 2021
…#196)

 - Data Table config parameter name is changed from "iDisplayLength" to "pageLength".
 - Streaming Queries navigation list, sources and sink tables config parameter "pageLength" is
   set to display all entries in it.  
 - Chart title changed from "Aggregation States" to "Aggregation State"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants