Skip to content

Fix broken inheritance from DataFrameColumn class (#7324) #7331

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Fix broken inheritance from DataFrameColumn class (#7324)
  • Loading branch information
asmirnov82 committed Dec 7, 2024
commit b351e330f222006064ecc1ec6702f5c98e80d9e0
2 changes: 1 addition & 1 deletion src/Microsoft.Data.Analysis/DataFrameColumn.cs
Original file line number Diff line number Diff line change
Expand Up @@ -460,7 +460,7 @@ public string ToString(long rowsToShow)
/// </summary>
/// <param name="ascending">Sorting order.</param>
/// <param name="putNullValuesLast">If true, null values are always put at the end.</param>
internal abstract PrimitiveDataFrameColumn<long> GetSortIndices(bool ascending, bool putNullValuesLast);
protected internal abstract PrimitiveDataFrameColumn<long> GetSortIndices(bool ascending, bool putNullValuesLast);

protected delegate long GetBufferSortIndex(int bufferIndex, int sortIndex);
protected delegate ValueTuple<T, int> GetValueAndBufferSortIndexAtBuffer<T>(int bufferIndex, int valueIndex);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -363,7 +363,7 @@ protected internal override Apache.Arrow.Array ToArrowArray(long startIndex, int
return new StringArray(numberOfRows, offsetsBuffer, dataBuffer, nullBuffer, nullCount, indexInBuffer);
}

internal override PrimitiveDataFrameColumn<long> GetSortIndices(bool ascending, bool putNullValuesLast) => throw new NotSupportedException();
protected internal override PrimitiveDataFrameColumn<long> GetSortIndices(bool ascending, bool putNullValuesLast) => throw new NotSupportedException();

public new ArrowStringDataFrameColumn Clone(long numberOfNullsToAppend = 0)
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,7 @@ public IEnumerator<string> GetEnumerator()
return (StringDataFrameColumn)base.Sort(ascending, putNullValuesLast);
}

internal override PrimitiveDataFrameColumn<long> GetSortIndices(bool ascending, bool putNullValuesLast)
protected internal override PrimitiveDataFrameColumn<long> GetSortIndices(bool ascending, bool putNullValuesLast)
{
var comparer = Comparer<string>.Default;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -384,6 +384,6 @@ protected override DataFrameColumn DropNullsImplementation()
return Clone();
}

internal override PrimitiveDataFrameColumn<long> GetSortIndices(bool ascending, bool putNullValuesLast) => throw new NotImplementedException();
protected internal override PrimitiveDataFrameColumn<long> GetSortIndices(bool ascending, bool putNullValuesLast) => throw new NotImplementedException();
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ public partial class PrimitiveDataFrameColumn<T> : DataFrameColumn
return (PrimitiveDataFrameColumn<T>)base.Sort(ascending, putNullValuesLast);
}

internal override PrimitiveDataFrameColumn<long> GetSortIndices(bool ascending = true, bool putNullValuesLast = true)
protected internal override PrimitiveDataFrameColumn<long> GetSortIndices(bool ascending = true, bool putNullValuesLast = true)
{
var comparer = Comparer<T>.Default;

Expand Down