Test local PR to break spotless#1
Open
niloc132 wants to merge 216 commits into
Open
Conversation
* Add PartitionedTable keys() method * Call view() when unique-keys is true for perf
* Add Treemap plotting Co-authored-by: Colin Alworth <colinalworth@deephaven.io> Co-authored-by: mikebender <mikebender@deephaven.io>
niloc132
force-pushed
the
2510-multiplexed-websocket
branch
2 times, most recently
from
808fd51 to
1cc40d8
Compare
Wrap PartitionedTableProxy
* Update redpanda docker repo Was getting an unauthorized error from docker.vectorized.io/redpanda: ``` ➜ main git:(heap-size-api) PATH=/usr/lib/jvm/java-11-openjdk-amd64/bin/:$PATH ./gradlew :Integrations:composeUp > Task :Integrations:composeUp redpanda uses an image, skipping Building with native build. Learn about native build in Compose here: https://docs.docker.com/go/compose-native-build/ Building with native build. Learn about native build in Compose here: https://docs.docker.com/go/compose-native-build/ Pulling redpanda (docker.vectorized.io/vectorized/redpanda:v21.9.5)... unauthorized: authentication required Removing network 7068f1a4e250e19015aa8e4c8be5964a_integrations__default > Task :Integrations:composeUp FAILED FAILURE: Build failed with an exception. ``` Check to use docker.redpanda.com/vectorized/redpanda instead, and now it works again.
That way you can get the port for the currently running server, which may be useful
* Added default import and updated javadoc (for updateBy) * Removed unneeded of() overload and updated tests Co-authored-by: Ryan Caudy <rcaudy@gmail.com>
…eephaven#2663) * Add wavgBy, wsumBy, absSumBy
Incremental step towards making deephaven#2671 possible.
* Bring C++ library to up to main branch * add missing copyrights, remove flatbuffers references * Delete alt-arrow. It has served its purpose * First cython wrapping for our C++ code; so far only wrapping types.h, intended to be extended with more sources/classes/code from here. * Improved README. Co-authored-by: Cristian Ferretti <cristianferretti@deephaven.io>
niloc132
force-pushed
the
2510-multiplexed-websocket
branch
from
ba25c9b to
a6e850b
Compare
- Needed to split up web versions, as we don't always publish 3 different packages with each version bump
- Wasn't taking into account subsecond patterns and time zones - Copied logic over from CsvTypeParser in Enterprise - Added additional short code time zones to match what is on Enterprise - Optimize by only changing the pattern if the existing pattern fails to parse - Fixes deephaven#2814
This removes the necessity for the `devroot` property to be set - it doesn't make sense for end-users. In code-generation or test cases where access to the source is still necessary, the appropriate system properties can be set as necessary.
* Reorganize code so that it can do an Arrow GetSchema call rather than doing a Get call and ignoring the results. * move some code to a new arrow_traits.h and arrow_util.h files * Code for demo chapter 3 * Respond to review feedback
Server should send a non-empty message when the session ends, and js client should notify user of failure in all cases. Partial deephaven#2985
* Update docker-compose files to use jetty image, remove use of `notebook` and `reverse-proxy` services * netty to jetty, also remove traefik label from all files * Add deephaven port and remove jetty from names, as there is no jetty suffix * Add port 6006 to TensorFlow and PyTorch examples for TensorBoard support * Add latest tag to examples containers
niloc132
added a commit
that referenced
this pull request
Jun 25, 2024
…eading schema at beginning of stream
niloc132
added a commit
that referenced
this pull request
Jun 25, 2024
…eading schema at beginning of stream
niloc132
added a commit
that referenced
this pull request
Jun 25, 2024
…eading schema at beginning of stream
niloc132
added a commit
that referenced
this pull request
Jun 25, 2024
…eading schema at beginning of stream
niloc132
added a commit
that referenced
this pull request
Jun 26, 2024
…eading schema at beginning of stream
niloc132
added a commit
that referenced
this pull request
Jun 26, 2024
…eading schema at beginning of stream
niloc132
added a commit
that referenced
this pull request
Jun 27, 2024
…eading schema at beginning of stream
niloc132
added a commit
that referenced
this pull request
Aug 2, 2024
niloc132
pushed a commit
that referenced
this pull request
Jul 11, 2025
…archicalTableViewSubscription snapshots. (deephaven#6984) This consists of three actual changes: 1. The HierarchicalTableView subscriptions would call snapshot, etc. from their update propagation job. This resulted in sorts that had no context. In the Core+ case, this meant we could not correctly log performance in the face of sorted rollups. This is the root of this entire PR. 2. The FunctionGeneratedTableFactory had a similar problem when refreshing a table. 3. I added a flag to PerformanceEntry that will fail when the auth context is not set. It defaults to false, to avoid breaking Core users unnecessarily. In Enterprise, we are breaking already; so this is an appropriate setting. I also added it to the tests, to find other cases where we were broken (like the FunctionGeneratedTableFactory). The rest of this is just changing the tests to include a context. I think #1 and #2 are a point release candidate, but not the rest of the fixes/changes. --------- Co-authored-by: Ryan Caudy <rcaudy@gmail.com>
niloc132
pushed a commit
that referenced
this pull request
Jul 11, 2025
…archicalTableViewSubscription snapshots. (deephaven#6999) Cherry-pick of deephaven#6984 This consists of three actual changes: 1. The HierarchicalTableView subscriptions would call snapshot, etc. from their update propagation job. This resulted in sorts that had no context. In the Core+ case, this meant we could not correctly log performance in the face of sorted rollups. This is the root of this entire PR. 2. The FunctionGeneratedTableFactory had a similar problem when refreshing a table. 3. I added a flag to PerformanceEntry that will fail when the auth context is not set. It defaults to false, to avoid breaking Core users unnecessarily. In Enterprise, we are breaking already; so this is an appropriate setting. I also added it to the tests, to find other cases where we were broken (like the FunctionGeneratedTableFactory). The rest of this is just changing the tests to include a context. I think #1 and #2 are a point release candidate, but not the rest of the fixes/changes. --------- Co-authored-by: Ryan Caudy <rcaudy@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
15 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.